diff options
author | Mark Dickinson <dickinsm@gmail.com> | 2009-03-18 08:25:36 (GMT) |
---|---|---|
committer | Mark Dickinson <dickinsm@gmail.com> | 2009-03-18 08:25:36 (GMT) |
commit | 7303b594e62349251cbb9d98141a96885fc88e6f (patch) | |
tree | f38c0d29fab1e3357b65fe9e1c2b91ea5f0a5bb8 /Lib | |
parent | 79f52039fe181d0f98b056b0b312a9bb42a59526 (diff) | |
download | cpython-7303b594e62349251cbb9d98141a96885fc88e6f.zip cpython-7303b594e62349251cbb9d98141a96885fc88e6f.tar.gz cpython-7303b594e62349251cbb9d98141a96885fc88e6f.tar.bz2 |
Merged revisions 70444 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk
........
r70444 | mark.dickinson | 2009-03-18 08:22:51 +0000 (Wed, 18 Mar 2009) | 3 lines
Fix bug in _insert_thousands_sep: too much zero padding could be
added for 'n' formats with non-repeating thousands-separator.
........
Diffstat (limited to 'Lib')
-rw-r--r-- | Lib/decimal.py | 3 | ||||
-rw-r--r-- | Lib/test/test_decimal.py | 22 |
2 files changed, 23 insertions, 2 deletions
diff --git a/Lib/decimal.py b/Lib/decimal.py index a751aa8..af29e68 100644 --- a/Lib/decimal.py +++ b/Lib/decimal.py @@ -5690,8 +5690,6 @@ def _insert_thousands_sep(digits, spec, min_width=1): groups = [] for l in _group_lengths(grouping): - if groups: - min_width -= len(sep) if l <= 0: raise ValueError("group length should be positive") # max(..., 1) forces at least 1 digit to the left of a separator @@ -5701,6 +5699,7 @@ def _insert_thousands_sep(digits, spec, min_width=1): min_width -= l if not digits and min_width <= 0: break + min_width -= len(sep) else: l = max(len(digits), min_width, 1) groups.append('0'*(l - len(digits)) + digits[-l:]) diff --git a/Lib/test/test_decimal.py b/Lib/test/test_decimal.py index c02b813..219348c 100644 --- a/Lib/test/test_decimal.py +++ b/Lib/test/test_decimal.py @@ -798,6 +798,28 @@ class DecimalFormatTest(unittest.TestCase): self.assertEqual(get_fmt(123456789, ru_RU, '.6n'), '1,23457e+8') self.assertEqual(get_fmt(123456789, crazy, '.6n'), '1&23457e+8') + # zero padding + self.assertEqual(get_fmt(1234, fr_FR, '03n'), '1234') + self.assertEqual(get_fmt(1234, fr_FR, '04n'), '1234') + self.assertEqual(get_fmt(1234, fr_FR, '05n'), '01234') + self.assertEqual(get_fmt(1234, fr_FR, '06n'), '001234') + + self.assertEqual(get_fmt(12345, en_US, '05n'), '12,345') + self.assertEqual(get_fmt(12345, en_US, '06n'), '12,345') + self.assertEqual(get_fmt(12345, en_US, '07n'), '012,345') + self.assertEqual(get_fmt(12345, en_US, '08n'), '0,012,345') + self.assertEqual(get_fmt(12345, en_US, '09n'), '0,012,345') + self.assertEqual(get_fmt(12345, en_US, '010n'), '00,012,345') + + self.assertEqual(get_fmt(123456, crazy, '06n'), '1-2345-6') + self.assertEqual(get_fmt(123456, crazy, '07n'), '1-2345-6') + self.assertEqual(get_fmt(123456, crazy, '08n'), '1-2345-6') + self.assertEqual(get_fmt(123456, crazy, '09n'), '01-2345-6') + self.assertEqual(get_fmt(123456, crazy, '010n'), '0-01-2345-6') + self.assertEqual(get_fmt(123456, crazy, '011n'), '0-01-2345-6') + self.assertEqual(get_fmt(123456, crazy, '012n'), '00-01-2345-6') + self.assertEqual(get_fmt(123456, crazy, '013n'), '000-01-2345-6') + class DecimalArithmeticOperatorsTest(unittest.TestCase): '''Unit tests for all arithmetic operators, binary and unary.''' |