From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Jeremy Drake <pgsql(at)jdrake(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: writing new regexp functions |
Date: | 2007-02-02 00:41:39 |
Message-ID: | 1875.1170376899@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Jeremy Drake <pgsql(at)jdrake(dot)com> writes:
> Is there some specific reason that these functions are static,
Yeah: not cluttering the global namespace. I'm not excited about
exporting everything that anybody could possibly want access to;
that just makes it harder to maintain the code. When you see a
static function, you know that you don't have to look further than
the current file to understand how it's used. When you see a global
function, the difficulty of knowing how it's used is an order of
magnitude higher, maybe more. What's more, if you want to change it
then you have to worry about the possible side-effects on unknown
non-core code that might be calling it.
Is there a reason for not putting your new code itself into regexp.c?
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Kate F | 2007-02-02 00:53:35 | Function proposal to find the type of a datum |
Previous Message | Koichi Suzuki | 2007-02-02 00:39:38 | Re: [HACKERS] Full page writes improvement |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2007-02-02 01:03:27 | Re: [HACKERS] Full page writes improvement |
Previous Message | Koichi Suzuki | 2007-02-02 00:39:38 | Re: [HACKERS] Full page writes improvement |