diff options
author | Mark Dickinson <dickinsm@gmail.com> | 2009-03-17 18:10:15 (GMT) |
---|---|---|
committer | Mark Dickinson <dickinsm@gmail.com> | 2009-03-17 18:10:15 (GMT) |
commit | ad41634313b291298935ff1b7044fc95d86da2be (patch) | |
tree | 3c4b8b0f7a473e68cc4c46b98b4cb71ff3dc3a2e | |
parent | 44c5481941fb9595c23d796b09dd9c34e0357fa5 (diff) | |
download | cpython-ad41634313b291298935ff1b7044fc95d86da2be.zip cpython-ad41634313b291298935ff1b7044fc95d86da2be.tar.gz cpython-ad41634313b291298935ff1b7044fc95d86da2be.tar.bz2 |
Merged revisions 70430 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk
........
r70430 | mark.dickinson | 2009-03-17 18:01:03 +0000 (Tue, 17 Mar 2009) | 3 lines
Fix bug in Decimal __format__ method that swapped left and right
alignment.
........
-rw-r--r-- | Lib/decimal.py | 4 | ||||
-rw-r--r-- | Lib/test/test_decimal.py | 6 | ||||
-rw-r--r-- | Misc/NEWS | 3 |
3 files changed, 11 insertions, 2 deletions
diff --git a/Lib/decimal.py b/Lib/decimal.py index 4b2f3f5..990afa7 100644 --- a/Lib/decimal.py +++ b/Lib/decimal.py @@ -5630,9 +5630,9 @@ def _format_align(body, spec_dict): align = spec_dict['align'] if align == '<': - result = padding + sign + body - elif align == '>': result = sign + body + padding + elif align == '>': + result = padding + sign + body elif align == '=': result = sign + padding + body else: #align == '^' diff --git a/Lib/test/test_decimal.py b/Lib/test/test_decimal.py index f40de8f..eaf9f0b 100644 --- a/Lib/test/test_decimal.py +++ b/Lib/test/test_decimal.py @@ -693,6 +693,12 @@ class DecimalFormatTest(unittest.TestCase): ('.0g', '-sNaN', '-sNaN'), ('', '1.00', '1.00'), + + # check alignment + ('<6', '123', '123 '), + ('>6', '123', ' 123'), + ('^6', '123', ' 123 '), + ('=+6', '123', '+ 123'), ] for fmt, d, result in test_values: self.assertEqual(format(Decimal(d), fmt), result) @@ -206,6 +206,9 @@ Core and Builtins Library ------- +- Fix Decimal.__format__ bug that swapped the meanings of the '<' and + '>' alignment characters. + - Issue #1222: locale.format() bug when the thousands separator is a space character. |