pgsql: Fix assorted bogosities in cash_in() and cash_out().

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix assorted bogosities in cash_in() and cash_out().
Date: 2011-10-29 18:32:20
Message-ID: E1RKDhc-00059Z-24@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix assorted bogosities in cash_in() and cash_out().

cash_out failed to handle multiple-byte thousands separators, as per bug
#6277 from Alexander Law. In addition, cash_in didn't handle that either,
nor could it handle multiple-byte positive_sign. Both routines failed to
support multiple-byte mon_decimal_point, which I did not think was worth
changing, but at least now they check for the possibility and fall back to
using '.' rather than emitting invalid output. Also, make cash_in handle
trailing negative signs, which formerly it would reject. Since cash_out
generates trailing negative signs whenever the locale tells it to, this
last omission represents a fail-to-reload-dumped-data bug. IMO that
justifies patching this all the way back.

Branch
------
REL8_3_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/b52ca458e9763442a0ece483c0529d50344557aa

Modified Files
--------------
src/backend/utils/adt/cash.c | 205 ++++++++++++++++++++++--------------------
1 files changed, 109 insertions(+), 96 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2011-10-29 18:47:05 pgsql: Update visibilitymap.c header comments.
Previous Message Tom Lane 2011-10-29 18:32:19 pgsql: Fix assorted bogosities in cash_in() and cash_out().