Re: writing new regexp functions

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

In response to

Responses

Browse pgsql-hackers by date

  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

Browse pgsql-patches by date

  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