pgsql: More zic cleanup. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: More zic cleanup.
Date
Msg-id E1c3Pen-0005n7-TL@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
More zic cleanup.

The workaround the IANA guys chose to get rid of the clang warning
we'd silenced in commit 23ed2ba81 turns out not to satisfy Coverity.
Go back to the previous solution, ie, remove the useless comparison
to SIZE_MAX.  (In principle, there could be machines out there where
it's not useless because ptrdiff_t is wider than size_t.  But the whole
thing is pretty academic anyway, as we could never approach this limit
for any sane estimate of the amount of data that zic will ever be asked
to work with.)

Also, s/lineno/lineno_t/g, because if we accept their decision to start
using "lineno" as a typedef, it is going to have very unpleasant
consequences in our next pgindent run.  Noted that while fooling with
pltcl yesterday.

Branch
------
REL9_3_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/3a8f24abdf9e4889ba83bf0ee1c3fe34a31a7f0e

Modified Files
--------------
src/timezone/zic.c | 23 +++++++++++------------
1 file changed, 11 insertions(+), 12 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: More zic cleanup.
Next
From: Tom Lane
Date:
Subject: pgsql: Fix silly nil-pointer-dereference bug introduced in commit d5f6f