commit 172bb39452ae8b3ccdf5d1f23ead46f44200cd49
author Serhiy Storchaka <storchaka@gmail.com> Sat Mar 30 08:33:02 2019 +0200
committer GitHub <noreply@github.com> Sat Mar 30 08:33:02 2019 +0200
tree 5e1effbca3664b839a81eb7a7d62fa4974cfbfb1
parent afbb7a371fb44edc731344eab5b474ad8f7b57d7

bpo-22831: Use "with" to avoid possible fd leaks in tools (part 2). (GH-10927)

Tools/scripts/cleanfuture.py

diff --git a/Tools/scripts/cleanfuture.py b/Tools/scripts/cleanfuture.py
index b48ab60..94f6912 100755
--- a/Tools/scripts/cleanfuture.py
+++ b/Tools/scripts/cleanfuture.py
@@ -96,11 +96,11 @@
         errprint("%r: I/O Error: %s" % (file, str(msg)))
         return
 
-    ff = FutureFinder(f, file)
-    changed = ff.run()
-    if changed:
-        ff.gettherest()
-    f.close()
+    with f:
+        ff = FutureFinder(f, file)
+        changed = ff.run()
+        if changed:
+            ff.gettherest()
     if changed:
         if verbose:
             print("changed.")
@@ -122,9 +122,8 @@
             os.rename(file, bak)
             if verbose:
                 print("renamed", file, "to", bak)
-            g = open(file, "w")
-            ff.write(g)
-            g.close()
+            with open(file, "w") as g:
+                ff.write(g)
             if verbose:
                 print("wrote new", file)
     else: