Re: GiST: memory allocation, cleanup - Mailing list pgsql-patches

From Tom Lane
Subject Re: GiST: memory allocation, cleanup
Date
Msg-id 23199.1099854353@sss.pgh.pa.us
Whole thread Raw
In response to Re: GiST: memory allocation, cleanup  (Neil Conway <neilc@samurai.com>)
List pgsql-patches
Neil Conway <neilc@samurai.com> writes:
> Tom Lane wrote:
>> I'm a bit dubious about this, mainly because you did not likewise
>> const-ify the other input arguments; it seems confusing to do a partial
>> const-ification.

> Well, "partial const-ification" is the rule rather than the exception in
> the backend right now. I'll take a look at adding more const qualifiers,
> but I don't really see why "partial const-ification" is confusing.

In this particular case I think it's confusing because the Datum and
nulls arrays are really two halves of a single data structure.
Const-ifying just one of them obscures that fact.  I'd be happy if you
marked both of them const.

            regards, tom lane

pgsql-patches by date:

Previous
From: Alin Vaida
Date:
Subject: New translation for 8.0: Romanian
Next
From: Alin Vaida
Date:
Subject: Romanian translation for 8.0: new file (psql)