From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
---|---|
To: | Alexander Korotkov <aekorotkov(at)gmail(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 15:43:01 |
Message-ID: | 1312818141-sup-4754@alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Excerpts from Alexander Korotkov's message of lun ago 08 06:27:33 -0400 2011:
> String-formatted relopts was never used before, but I've used it in
> buffering GiST index build patch and encountered with following compiler
> warnings:
>
> reloptions.c:259: warning: initializer-string for array of chars is too long
> reloptions.c:259: warning: (near initialization for
> ‘stringRelOpts[0].default_val’**)
>
> It is caused by definition of default field of relopt_string structure as
> 1-length character array. This seems to be a design flaw in the reloptions.c
> code. Any thoughts?
Maybe this needs to use the new FLEXIBLE_ARRAY_MEMBER stuff. Can you try that please?
--
Álvaro Herrera <alvherre(at)commandprompt(dot)com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support
From | Date | Subject | |
---|---|---|---|
Next Message | Alexander Korotkov | 2011-08-08 15:50:53 | Re: Compiler warnings with stringRelOpts (was WIP: Fast GiST index build) |
Previous Message | Florian Pflug | 2011-08-08 15:35:40 | Re: WIP fix proposal for bug #6123 |