]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix assorted breakage in to_char()'s OF format option.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 17 Mar 2016 19:50:33 +0000 (15:50 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 17 Mar 2016 19:50:33 +0000 (15:50 -0400)
commitd0a585c6997de6aebd5866c9c1791a1badfcee0a
tree552fa50034ca039c47a412de1bd88a19130bbbdc
parent8ee2e5f78cd7f99d41ddf2c396d83adb6f9df2db
Fix assorted breakage in to_char()'s OF format option.

In HEAD, fix incorrect field width for hours part of OF when tm_gmtoff is
negative.  This was introduced by commit 2d87eedc1d4468d3 as a result of
falsely applying a pattern that's correct when + signs are omitted, which
is not the case for OF.

In 9.4, fix missing abs() call that allowed a sign to be attached to the
minutes part of OF.  This was fixed in 9.5 by 9b43d73b3f9bef27, but for
inscrutable reasons not back-patched.

In all three versions, ensure that the sign of tm_gmtoff is correctly
reported even when the GMT offset is less than 1 hour.

Add regression tests, which evidently we desperately need here.

Thomas Munro and Tom Lane, per report from David Fetter
src/backend/utils/adt/formatting.c
src/test/regress/expected/timestamptz.out
src/test/regress/sql/timestamptz.sql