Fix ResourceWarning: unclosed file

Also uniformize usage of the 'with' context manager to prevent resource leaks.

Signed-off-by: Mickaël Schoentgen <[e-mail]>

IPython/core/crashhandler.py

@@ -179,13 +179,14 b' class CrashHandler(object):'
             print('Could not create crash report on disk.', file=sys.stderr)
             return
 
+        with report:
         # Inform user on stderr of what happened
         print('\n'+'*'*70+'\n', file=sys.stderr)
         print(self.message_template.format(**self.info), file=sys.stderr)
 
         # Construct report on disk
         report.write(self.make_report(traceback))
-        report.close()
+
         input("Hit <Enter> to quit (your terminal may close):")
 
     def make_report(self,traceback):

IPython/core/debugger.py

@@ -206,9 +206,8 b' def _file_lines(fname):'
     except IOError:
         return []
     else:
-        out = outfile.readlines()
-        outfile.close()
-        return out
+        with out:
+            return outfile.readlines()
 
 
 class Pdb(OldPdb):

IPython/core/interactiveshell.py

@@ -3469,9 +3469,8 b' class InteractiveShell(SingletonConfigurable):'
         self.tempfiles.append(filename)
 
         if data:
-            tmp_file = open(filename,'w')
+            with open(filename, 'w') as tmp_file:
             tmp_file.write(data)
-            tmp_file.close()
         return filename
 
     @undoc

IPython/core/magics/code.py

@@ -722,7 +722,8 b' class CodeMagics(Magics):'
 
         if is_temp:
             try:
-                return open(filename).read()
+                with open(filename) as f:
+                    return f.read()
             except IOError as msg:
                 if msg.filename == filename:
                     warn('File not found. Did you forget to save?')

IPython/core/magics/execution.py

@@ -370,9 +370,8 b' python-profiler package from non-free.""")'
             print('\n*** Profile stats marshalled to file',\
                   repr(dump_file)+'.',sys_exit)
         if text_file:
-            pfile = open(text_file,'w')
+            with open(text_file, 'w') as pfile:
             pfile.write(output)
-            pfile.close()
             print('\n*** Profile printout saved to text file',\
                   repr(text_file)+'.',sys_exit)
 

IPython/core/magics/packaging.py

@@ -35,7 +35,8 b' def _get_conda_executable():'
     # Otherwise, attempt to extract the executable from conda history.
     # This applies in any conda environment.
     R = re.compile(r"^#\s*cmd:\s*(?P<command>.*conda)\s[create|install]")
-    for line in open(os.path.join(sys.prefix, 'conda-meta', 'history')):
+    with open(os.path.join(sys.prefix, 'conda-meta', 'history')) as f:
+        for line in f:
         match = R.match(line)
         if match:
             return match.groupdict()['command']

IPython/extensions/storemagic.py

@@ -172,11 +172,11 b' class StoreMagics(Magics):'
                     fil = open(fnam, 'a')
                 else:
                     fil = open(fnam, 'w')
+                with fil:
                 obj = ip.ev(args[0])
                 print("Writing '%s' (%s) to file '%s'." % (args[0],
                   obj.__class__.__name__, fnam))
 
-
                 if not isinstance (obj, str):
                     from pprint import pprint
                     pprint(obj, fil)
@@ -185,7 +185,6 b' class StoreMagics(Magics):'
                     if not obj.endswith('\n'):
                         fil.write('\n')
 
-                fil.close()
                 return
 
             # %store foo

IPython/extensions/tests/test_autoreload.py

@@ -109,19 +109,13 b' class Fixture(object):'
         time.sleep(1.05)
 
         # Write
-        f = open(filename, 'w')
-        try:
+        with open(filename, 'w') as f:
             f.write(content)
-        finally:
-            f.close()
 
     def new_module(self, code):
         mod_name, mod_fn = self.get_module()
-        f = open(mod_fn, 'w')
-        try:
+        with open(mod_fn, 'w') as f:
             f.write(code)
-        finally:
-            f.close()
         return mod_name, mod_fn
 
 #-----------------------------------------------------------------------------

IPython/testing/plugin/ipdoctest.py

@@ -688,11 +688,8 b' class ExtensionDoctest(doctests.Doctest):'
         else:
             if self.extension and anyp(filename.endswith, self.extension):
                 name = os.path.basename(filename)
-                dh = open(filename)
-                try:
+                with open(filename) as dh:
                     doc = dh.read()
-                finally:
-                    dh.close()
                 test = self.parser.get_doctest(
                     doc, globs={'__file__': filename}, name=name,
                     filename=filename, lineno=0)

IPython/testing/tools.py

@@ -422,8 +422,7 b' def mute_warn():'
 def make_tempfile(name):
     """ Create an empty, named, temporary file for the duration of the context.
     """
-    f = open(name, 'w')
-    f.close()
+    open(name, 'w').close()
     try:
         yield
     finally:

IPython/utils/tests/test_module_paths.py

@@ -37,8 +37,7 b" TMP_TEST_DIR = tempfile.mkdtemp(suffix='with.dot')"
 old_syspath = sys.path
 
 def make_empty_file(fname):
-    f = open(fname, 'w')
-    f.close()
+    open(fname, 'w').close()
 
 
 def setup():

IPython/utils/tests/test_openpy.py

@@ -8,7 +8,7 b' mydir = os.path.dirname(__file__)'
 nonascii_path = os.path.join(mydir, '../../core/tests/nonascii.py')
 
 def test_detect_encoding():
-    f = open(nonascii_path, 'rb')
+    with open(nonascii_path, 'rb') as f:
     enc, lines = openpy.detect_encoding(f.readline)
     nt.assert_equal(enc, 'iso-8859-5')
 

docs/sphinxext/apigen.py

@@ -392,9 +392,8 b' class ApiDocWriter(object):'
             # write out to file
             outfile = os.path.join(outdir,
                                    m + self.rst_extension)
-            fileobj = open(outfile, 'wt')
+            with open(outfile, 'wt') as fileobj:
             fileobj.write(api_str)
-            fileobj.close()
             written_modules.append(m)
         self.written_modules = written_modules
 
@@ -445,11 +444,10 b' class ApiDocWriter(object):'
             relpath = outdir.replace(relative_to + os.path.sep, '')
         else:
             relpath = outdir
-        idx = open(path,'wt')
+        with open(path,'wt') as idx:
         w = idx.write
         w('.. AUTO-GENERATED FILE -- DO NOT EDIT!\n\n')
         w('.. autosummary::\n'
           '   :toctree: %s\n\n' % relpath)
         for mod in self.written_modules:
             w('   %s\n' % mod)
-        idx.close()

examples/IPython Kernel/Rich Output.ipynb

@@ -3023,7 +3023,8 b''
    "source": [
     "from IPython.display import HTML\n",
     "from base64 import b64encode\n",
-    "video = open(\"../images/animation.m4v\", \"rb\").read()\n",
+    "with open(\"../images/animation.m4v\", \"rb\") as f:\n",
+    "    video = f.read()\n",
     "video_encoded = b64encode(video).decode('ascii')\n",
     "video_tag = '<video controls alt=\"test\" src=\"data:video/x-m4v;base64,{0}\">'.format(video_encoded)\n",
     "HTML(data=video_tag)"

examples/IPython Kernel/ipython-get-history.py

@@ -27,6 +27,8 b' if len(sys.argv) > 2:'
 else:
     dest = sys.stdout
     raw = True
+
+with dest:
 dest.write("# coding: utf-8\n")
 
 # Profiles other than 'default' can be specified here with a profile= argument: