pgsql: Add missing format attributes

Started by Peter Eisentrautover 14 years ago3 messages
#1Peter Eisentraut
peter_e@gmx.net

Add missing format attributes

Add __attribute__ decorations for printf format checking to the places that
were missing them. Fix the resulting warnings. Add
-Wmissing-format-attribute to the standard set of warnings for GCC, so these
don't happen again.

The warning fixes here are relatively harmless. The one serious problem
discovered by this was already committed earlier in
cf15fb5cabfbc71e07be23cfbc813daee6c5014f.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/52ce20589a8bac4eccaea043b1fe283daaf4f9e3

Modified Files
--------------
configure | 60 ++++++++++++++++++++++++++++++
configure.in | 1 +
contrib/pg_upgrade/info.c | 2 +-
contrib/pg_upgrade/pg_upgrade.h | 16 +++++---
contrib/pg_upgrade/relfilenode.c | 4 +-
contrib/pgcrypto/px.h | 3 +-
src/bin/pg_dump/pg_backup_archiver.c | 4 +-
src/bin/pg_dump/pg_backup_tar.c | 2 +-
src/include/lib/stringinfo.h | 3 +-
src/interfaces/ecpg/ecpglib/descriptor.c | 4 +-
src/interfaces/ecpg/ecpglib/error.c | 2 +-
src/interfaces/ecpg/ecpglib/execute.c | 2 +-
src/interfaces/ecpg/ecpglib/extern.h | 2 +-
13 files changed, 87 insertions(+), 18 deletions(-)

#2Fujii Masao
masao.fujii@gmail.com
In reply to: Peter Eisentraut (#1)
Re: [COMMITTERS] pgsql: Add missing format attributes

On Sun, Sep 11, 2011 at 5:17 AM, Peter Eisentraut <peter_e@gmx.net> wrote:

Add missing format attributes

Add __attribute__ decorations for printf format checking to the places that
were missing them.  Fix the resulting warnings.  Add
-Wmissing-format-attribute to the standard set of warnings for GCC, so these
don't happen again.

The warning fixes here are relatively harmless.  The one serious problem
discovered by this was already committed earlier in
cf15fb5cabfbc71e07be23cfbc813daee6c5014f.

This commit causes the following warning at the compile time.

error.c: In function 'ecpg_raise_backend':
error.c:339: warning: field precision should have type 'int', but
argument 2 has type 'long unsigned int'

Regards,

--
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center

#3Peter Eisentraut
peter_e@gmx.net
In reply to: Fujii Masao (#2)
Re: [COMMITTERS] pgsql: Add missing format attributes

On sön, 2011-09-11 at 16:11 +0900, Fujii Masao wrote:

On Sun, Sep 11, 2011 at 5:17 AM, Peter Eisentraut <peter_e@gmx.net> wrote:

Add missing format attributes

Add __attribute__ decorations for printf format checking to the places that
were missing them. Fix the resulting warnings. Add
-Wmissing-format-attribute to the standard set of warnings for GCC, so these
don't happen again.

The warning fixes here are relatively harmless. The one serious problem
discovered by this was already committed earlier in
cf15fb5cabfbc71e07be23cfbc813daee6c5014f.

This commit causes the following warning at the compile time.

error.c: In function 'ecpg_raise_backend':
error.c:339: warning: field precision should have type 'int', but
argument 2 has type 'long unsigned int'

Fixed, thanks.