Re: Fixes for compiler warnings - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: Fixes for compiler warnings
Date
Msg-id 200901180134.10251.peter_e@gmx.net
Whole thread Raw
In response to Fixes for compiler warnings  (Alan Li <alanwli@gmail.com>)
Responses Re: Fixes for compiler warnings  (Gregory Stark <stark@enterprisedb.com>)
List pgsql-hackers
On Saturday 17 January 2009 11:44:07 Alan Li wrote:
> Attached are patches to fix the following compiler warnings that I see when
> using gcc 4.3.2.
>
> MASTER warning:
> tablecmds.c: In function 'DropErrorMsgWrongType':
> tablecmds.c:601: warning: format not a string literal and no format
> arguments
>
> REL8_3_STABLE warnings:
> utility.c: In function 'DropErrorMsgWrongType':
> utility.c:129: warning: format not a string literal and no format arguments
> trigger.c: In function 'ConvertTriggerToFK':
> trigger.c:600: warning: format not a string literal and no format arguments
> trigger.c:616: warning: format not a string literal and no format arguments
> trigger.c:628: warning: format not a string literal and no format arguments
> guc.c: In function 'set_config_option':
> guc.c:4424: warning: format not a string literal and no format arguments
> describe.c: In function 'describeOneTableDetails':
> describe.c:1294: warning: format not a string literal and no format
> arguments

You apparently have your compiler configured with -Wformat-security.  Our code 
doesn't do that.  I think the cases the warning complains about are fine and 
the way the warning is designed is a bit bogus.


pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: Statement-level triggers and inheritance
Next
From: Gregory Stark
Date:
Subject: Re: Fixes for compiler warnings