Directly sum up lengths

sebastinas · sebastinas · commit 2776156c5353 · 2022-05-05T00:21:35.000+02:00
There is no need to build the complete string.
diff --git a/bpython/curtsiesfrontend/repl.py b/bpython/curtsiesfrontend/repl.py
@@ -1536,15 +1536,15 @@ def number_of_padding_chars_on_current_cursor_line(self):
 
         Should return zero unless there are fullwidth characters."""
         full_line = self.current_cursor_line_without_suggestion
-        line_with_padding = "".join(
-            line.s
+        line_with_padding_len = sum(
+            len(line.s)
             for line in paint.display_linize(
                 self.current_cursor_line_without_suggestion.s, self.width
             )
         )
 
         # the difference in length here is how much padding there is
-        return len(line_with_padding) - len(full_line)
+        return line_with_padding_len - len(full_line)
 
     def paint(
         self,
