Partially apply ruff/flake8-simplify rule SIM115

SIM115 Use a context manager for opening files

Author: DimitriPapadopoulos

src/cffi/recompiler.py

@@ -287,9 +287,8 @@ def write_source_to_f(self, f, preamble):
             self.write_c_source_to_f(f, preamble)
 
     def _rel_readlines(self, filename):
-        g = open(os.path.join(os.path.dirname(__file__), filename), 'r')
-        lines = g.readlines()
-        g.close()
+        with open(os.path.join(os.path.dirname(__file__), filename), 'r') as g:
+            lines = g.readlines()
         return lines
 
     def write_c_source_to_f(self, f, preamble):

testing/cffi0/test_verify.py

@@ -38,9 +38,8 @@ def setup_module():
     def _write_source_and_check(self, file=None):
         base_write_source(self, file)
         if file is None:
-            f = open(self.sourcefilename)
-            data = f.read()
-            f.close()
+            with open(self.sourcefilename) as f:
+                data = f.read()
             data = _r_comment.sub(' ', data)
             data = _r_string.sub('"skipped"', data)
             assert '$' not in data
@@ -1440,9 +1439,8 @@ def test_relative_to():
     tmpdir = tempfile.mkdtemp(dir=str(udir))
     ffi = FFI()
     ffi.cdef("int foo(int);")
-    f = open(os.path.join(tmpdir, 'foo.h'), 'w')
-    f.write("int foo(int a) { return a + 42; }\n")
-    f.close()
+    with open(os.path.join(tmpdir, 'foo.h'), 'w') as f:
+        f.write("int foo(int a) { return a + 42; }\n")
     lib = ffi.verify('#include "foo.h"',
                      include_dirs=['.'],
                      relative_to=os.path.join(tmpdir, 'x'))

testing/cffi1/test_verify1.py

@@ -1410,9 +1410,8 @@ def test_relative_to():
     tmpdir = tempfile.mkdtemp(dir=str(udir))
     ffi = FFI()
     ffi.cdef("int foo(int);")
-    f = open(os.path.join(tmpdir, 'foo.h'), 'w')
-    f.write("int foo(int a) { return a + 42; }\n")
-    f.close()
+    with open(os.path.join(tmpdir, 'foo.h'), 'w') as f:
+        f.write("int foo(int a) { return a + 42; }\n")
     lib = ffi.verify('#include "foo.h"',
                      include_dirs=['.'],
                      relative_to=os.path.join(tmpdir, 'x'))