summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMark Dickinson <dickinsm@gmail.com>2009-03-17 18:10:15 (GMT)
committerMark Dickinson <dickinsm@gmail.com>2009-03-17 18:10:15 (GMT)
commitad41634313b291298935ff1b7044fc95d86da2be (patch)
tree3c4b8b0f7a473e68cc4c46b98b4cb71ff3dc3a2e
parent44c5481941fb9595c23d796b09dd9c34e0357fa5 (diff)
downloadcpython-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.py4
-rw-r--r--Lib/test/test_decimal.py6
-rw-r--r--Misc/NEWS3
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)
diff --git a/Misc/NEWS b/Misc/NEWS
index afeb987..bae6e78 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -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.