better cleaning tool

Akuli · Akuli · commit b90e4de00c13 · 2017-01-24T22:46:25.000+02:00
diff --git a/basics/handy-stuff-strings.md b/basics/handy-stuff-strings.md
@@ -400,7 +400,7 @@ ValueError: could not convert string to float: 'hello'
     string.
 - `len(string)` returns string's length.
 - We can use `str`, `int` and `float` to convert values to different
-	types.
+    types.
 
 ## Exercises
 
diff --git a/basics/what-is-true.md b/basics/what-is-true.md
@@ -203,13 +203,13 @@ if value is None: ...  # best
 ## Summary
 
 - `if thing:` does the same thing as `if bool(thing):`. This also
-	works with while loops and most other things that are usually used
-	with Booleans.
+    works with while loops and most other things that are usually used
+    with Booleans.
 - `bool()` of most things is True, but `bool()` values of None,
-	zero and most empty things are False.
+    zero and most empty things are False.
 - Use `is` and `is not` when comparing to None, `==` and `!=` when
-	checking if a number is zero and rely on the Boolean value
-	when checking if something is empty.
+    checking if a number is zero and rely on the Boolean value
+    when checking if something is empty.
 
 ***
 
diff --git a/strip.py b/strip.py
@@ -31,12 +31,15 @@
 import common
 
 
+def fix(line):
+    return line.rstrip().expandtabs(4)
+
+
 def needs_stripping(file):
     with common.slashfix_open(file, 'r') as f:
         for line in f:
             line = line.rstrip('\n')
-            if line != line.rstrip():
-                # contains trailing whitespace other than '\n'
+            if line != fix(line):
                 return True
     return False
 
@@ -45,7 +48,11 @@ def strip(file):
     lines = []
     with common.slashfix_open(file, 'r') as f:
         for line in f:
-            lines.append(line.rstrip())
+            line = line.rstrip('\n')
+            # it's important to do as much as possible here because the
+            # file may be lost if writing fails, that's why fix() is
+            # here
+            lines.append(fix(line))
     with common.slashfix_open(file, 'w') as f:
         for line in lines:
             print(line, file=f)
@@ -57,7 +64,7 @@ def main():
             print("Stripping", file)
             strip(file)
         else:
-            print("No trailing whitespace in", file)
+            print("No trailing whitespace or tabs in", file)
 
 
 if __name__ == '__main__':
