Merge pull request #1255 from correctmost/cm/combine-rstrips

Combine rstrip calls in trailing_whitespace check
PyCQA · Aug 9, 2024 · c8e36a0 · c8e36a0
2 parents fc2c2ba + 035b52f
commit c8e36a0
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/pycodestyle.py b/pycodestyle.py
@@ -233,9 +233,11 @@ def trailing_whitespace(physical_line):
     W291: spam(1) \n#
     W293: class Foo(object):\n    \n    bang = 12
     """
-    physical_line = physical_line.rstrip('\n')    # chr(10), newline
-    physical_line = physical_line.rstrip('\r')    # chr(13), carriage return
-    physical_line = physical_line.rstrip('\x0c')  # chr(12), form feed, ^L
+    # Strip these trailing characters:
+    # - chr(10), newline
+    # - chr(13), carriage return
+    # - chr(12), form feed, ^L
+    physical_line = physical_line.rstrip('\n\r\x0c')
     stripped = physical_line.rstrip(' \t\v')
     if physical_line != stripped:
         if stripped: