From: | Alexander Korotkov <aekorotkov(at)gmail(dot)com> |
---|---|
To: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
Cc: | Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Compiler warnings with stringRelOpts (was WIP: Fast GiST index build) |
Date: | 2011-08-08 17:21:17 |
Message-ID: | CAPpHfdv9v82BxNkRCt8tGmc_6SAj0zP-uB6h0PcCBaVKQbB_VA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Aug 8, 2011 at 8:27 PM, Alvaro Herrera
<alvherre(at)commandprompt(dot)com>wrote:
> An array of relopt_string? Isn't that a bit strange? If I recall
> correctly, the point of this was to be able to allocate the
> relopt_string struct and the char array itself as a single palloc unit,
> in a single call somewhere in the reloptions API (which was convoluted
> in some points precisely to let the string case work). I don't have the
> details of this fresh in my mind though. It certainly worked with more
> than one string option when I committed it, IIRC.
>
Yes, it seems strange. But it also seems that both were added by your commit
of table-based parser:
http://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=ba748f7a11ef884277b61d1708a17a44acfd1736
------
With best regards,
Alexander Korotkov.
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2011-08-08 17:23:29 | Re: [RFC] Common object property boards |
Previous Message | Robert Haas | 2011-08-08 17:19:13 | Re: fstat vs. lseek |