]> granicus.if.org Git - postgresql/commit
Allow use of "z" flag in our printf calls, and use it where appropriate.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 23 Jan 2014 22:18:23 +0000 (17:18 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 23 Jan 2014 22:18:33 +0000 (17:18 -0500)
commitac4ef637ad2ff2a24847f67d14027b8745f6741e
treecb49061503740d1bd9a0c916df55a29b4973fa64
parentec8f692c3cd5760435712b7ec4afa8f014ed7b2e
Allow use of "z" flag in our printf calls, and use it where appropriate.

Since C99, it's been standard for printf and friends to accept a "z" size
modifier, meaning "whatever size size_t has".  Up to now we've generally
dealt with printing size_t values by explicitly casting them to unsigned
long and using the "l" modifier; but this is really the wrong thing on
platforms where pointers are wider than longs (such as Win64).  So let's
start using "z" instead.  To ensure we can do that on all platforms, teach
src/port/snprintf.c to understand "z", and add a configure test to force
use of that implementation when the platform's version doesn't handle "z".

Having done that, modify a bunch of places that were using the
unsigned-long hack to use "z" instead.  This patch doesn't pretend to have
gotten everyplace that could benefit, but it catches many of them.  I made
an effort in particular to ensure that all uses of the same error message
text were updated together, so as not to increase the number of
translatable strings.

It's possible that this change will result in format-string warnings from
pre-C99 compilers.  We might have to reconsider if there are any popular
compilers that will warn about this; but let's start by seeing what the
buildfarm thinks.

Andres Freund, with a little additional work by me
27 files changed:
config/c-library.m4
configure
configure.in
src/backend/access/common/indextuple.c
src/backend/access/gin/ginentrypage.c
src/backend/access/hash/hashinsert.c
src/backend/access/heap/hio.c
src/backend/access/heap/rewriteheap.c
src/backend/access/nbtree/nbtinsert.c
src/backend/access/nbtree/nbtsort.c
src/backend/access/spgist/spgdoinsert.c
src/backend/access/spgist/spgutils.c
src/backend/access/transam/xlog.c
src/backend/nodes/readfuncs.c
src/backend/port/sysv_shmem.c
src/backend/port/win32_shmem.c
src/backend/storage/file/fd.c
src/backend/storage/freespace/freespace.c
src/backend/storage/ipc/dsm.c
src/backend/storage/ipc/dsm_impl.c
src/backend/storage/ipc/ipci.c
src/backend/storage/ipc/shmem.c
src/backend/storage/lmgr/predicate.c
src/backend/utils/mmgr/aset.c
src/backend/utils/mmgr/mcxt.c
src/include/pg_config.h.in
src/port/snprintf.c