diff options
author | Eric V. Smith <eric@trueblade.com> | 2015-09-29 14:30:47 (GMT) |
---|---|---|
committer | Eric V. Smith <eric@trueblade.com> | 2015-09-29 14:30:47 (GMT) |
commit | 6dcada3bcfbaa35930a55bf67e5d6393f444f68a (patch) | |
tree | 729ab8053604716c5323b138a1cf824a4d888714 | |
parent | 2fbcd2a057bb09c385d90386e4c17038360b0611 (diff) | |
parent | ad4003c7fb9072c2697480ee84440408f7a49cd9 (diff) | |
download | cpython-6dcada3bcfbaa35930a55bf67e5d6393f444f68a.zip cpython-6dcada3bcfbaa35930a55bf67e5d6393f444f68a.tar.gz cpython-6dcada3bcfbaa35930a55bf67e5d6393f444f68a.tar.bz2 |
Issue #25034: Merge from 3.5.
-rw-r--r-- | Lib/string.py | 11 | ||||
-rw-r--r-- | Lib/test/test_string.py | 2 | ||||
-rw-r--r-- | Misc/ACKS | 1 | ||||
-rw-r--r-- | Misc/NEWS | 3 |
4 files changed, 12 insertions, 5 deletions
diff --git a/Lib/string.py b/Lib/string.py index e7b692d..1add44b 100644 --- a/Lib/string.py +++ b/Lib/string.py @@ -183,7 +183,7 @@ class Formatter: def vformat(self, format_string, args, kwargs): used_args = set() - result = self._vformat(format_string, args, kwargs, used_args, 2) + result, _ = self._vformat(format_string, args, kwargs, used_args, 2) self.check_unused_args(used_args, args, kwargs) return result @@ -230,14 +230,15 @@ class Formatter: obj = self.convert_field(obj, conversion) # expand the format spec, if needed - format_spec = self._vformat(format_spec, args, kwargs, - used_args, recursion_depth-1, - auto_arg_index=auto_arg_index) + format_spec, auto_arg_index = self._vformat( + format_spec, args, kwargs, + used_args, recursion_depth-1, + auto_arg_index=auto_arg_index) # format the object and append to the result result.append(self.format_field(obj, format_spec)) - return ''.join(result) + return ''.join(result), auto_arg_index def get_value(self, key, args, kwargs): diff --git a/Lib/test/test_string.py b/Lib/test/test_string.py index 5d37e16..0cd2b86 100644 --- a/Lib/test/test_string.py +++ b/Lib/test/test_string.py @@ -58,6 +58,8 @@ class ModuleTest(unittest.TestCase): 'foo{1}{num}{1}'.format(None, 'bar', num=6)) self.assertEqual(fmt.format('{:^{}}', 'bar', 6), '{:^{}}'.format('bar', 6)) + self.assertEqual(fmt.format('{:^{}} {}', 'bar', 6, 'X'), + '{:^{}} {}'.format('bar', 6, 'X')) self.assertEqual(fmt.format('{:^{pad}}{}', 'foo', 'bar', pad=6), '{:^{pad}}{}'.format('foo', 'bar', pad=6)) @@ -1013,6 +1013,7 @@ Tony Nelson Trent Nelson Chad Netzer Max Neunhöffer +Anthon van der Neut George Neville-Neil Hieu Nguyen Johannes Nicolai @@ -190,6 +190,9 @@ Core and Builtins Library ------- +- Issue #25034: Fix string.Formatter problem with auto-numbering and + nested format_specs. Patch by Anthon van der Neut. + - Issue #25233: Rewrite the guts of asyncio.Queue to be more understandable and correct. - Issue #23600: Default implementation of tzinfo.fromutc() was returning |