react to feedback

Author: xi

Doc/library/textwrap.rst

@@ -286,6 +286,8 @@ hyphenated words; only then will long words be broken if necessary, unless
       (default: ``len``) Used to determine the length of a string. You can
       provide a custom function, e.g. to account for wide characters.
 
+      .. versionadded:: 3.11
+
 
    .. index:: single: ...; placeholder
 

Lib/test/test_textwrap.py

@@ -1078,22 +1078,19 @@ def test_first_word_too_long_but_placeholder_fits(self):
 
 
 class WideCharacterTestCase(BaseTestCase):
-    def setUp(self):
+    def test_wide_character(self):
         def text_len(text):
             n = 0
             for c in text:
-                if unicodedata.east_asian_width(c) in ['F', 'W']:
+                if unicodedata.east_asian_width(c) in {'F', 'W'}:
                     n += 2
                 else:
                     n += 1
             return n
 
-        self.wrapper = TextWrapper(width=5, text_len=text_len)
-
-    def test_wide_character(self):
         text = "123 🔧"
-        result = self.wrapper.wrap(text, **kwargs)
-        self.check(result, ["123", "🔧"])
+        expected = ["123", "🔧"]
+        self.check_wrap(text, 6, expected, text_len=text_len)
 
 
 if __name__ == '__main__':

Lib/textwrap.py

@@ -261,8 +261,8 @@ def _wrap_chunks(self, chunks):
                 indent = self.subsequent_indent
             else:
                 indent = self.initial_indent
-            if self.text_len(indent) +
-                    self.text_len(self.placeholder.lstrip()) > self.width:
+            if (self.text_len(indent) +
+                    self.text_len(self.placeholder.lstrip()) > self.width):
                 raise ValueError("placeholder too large for max width")
 
         # Arrange in reverse order so items can be efficiently popped