diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2015-10-28 19:39:36 (GMT) |
---|---|---|
committer | Serhiy Storchaka <storchaka@gmail.com> | 2015-10-28 19:39:36 (GMT) |
commit | ea4cb63e68d96697ff8eb9ea86da6158f27b95a2 (patch) | |
tree | 017471658ea2343cce55ac59a16926db4c0b60e1 /Lib | |
parent | 68b6874f59d80c41785a4d4a6874be89ad08fcd5 (diff) | |
download | cpython-ea4cb63e68d96697ff8eb9ea86da6158f27b95a2.zip cpython-ea4cb63e68d96697ff8eb9ea86da6158f27b95a2.tar.gz cpython-ea4cb63e68d96697ff8eb9ea86da6158f27b95a2.tar.bz2 |
Issue #21827: Fixed textwrap.dedent() for the case when largest common
whitespace is a substring of smallest leading whitespace.
Based on patch by Robert Li.
Diffstat (limited to 'Lib')
-rw-r--r-- | Lib/test/test_textwrap.py | 5 | ||||
-rw-r--r-- | Lib/textwrap.py | 12 |
2 files changed, 13 insertions, 4 deletions
diff --git a/Lib/test/test_textwrap.py b/Lib/test/test_textwrap.py index 1bba77e..ea7c089 100644 --- a/Lib/test/test_textwrap.py +++ b/Lib/test/test_textwrap.py @@ -732,6 +732,11 @@ def foo(): expect = "hello there\n how are you?" self.assertEqual(expect, dedent(text)) + # test margin is smaller than smallest indent + text = " \thello there\n \thow are you?\n \tI'm fine, thanks" + expect = " \thello there\n \thow are you?\n\tI'm fine, thanks" + self.assertEqual(expect, dedent(text)) + # Test textwrap.indent class IndentTestCase(unittest.TestCase): diff --git a/Lib/textwrap.py b/Lib/textwrap.py index 58867f9..1759b0d 100644 --- a/Lib/textwrap.py +++ b/Lib/textwrap.py @@ -429,11 +429,15 @@ def dedent(text): elif margin.startswith(indent): margin = indent - # Current line and previous winner have no common whitespace: - # there is no margin. + # Find the largest common whitespace between current line and previous + # winner. else: - margin = "" - break + for i, (x, y) in enumerate(zip(margin, indent)): + if x != y: + margin = margin[:i] + break + else: + margin = margin[:len(indent)] # sanity check (testing/debugging only) if 0 and margin: |