Commit 41b95df3 authored by Benjamin Peterson's avatar Benjamin Peterson

close files explicit (closes #21610)

Patch by mattip.
parent 738cc741
...@@ -109,14 +109,14 @@ class ModuleFinder: ...@@ -109,14 +109,14 @@ class ModuleFinder:
def run_script(self, pathname): def run_script(self, pathname):
self.msg(2, "run_script", pathname) self.msg(2, "run_script", pathname)
fp = open(pathname, READ_MODE) with open(pathname, READ_MODE) as fp:
stuff = ("", "r", imp.PY_SOURCE) stuff = ("", "r", imp.PY_SOURCE)
self.load_module('__main__', fp, pathname, stuff) self.load_module('__main__', fp, pathname, stuff)
def load_file(self, pathname): def load_file(self, pathname):
dir, name = os.path.split(pathname) dir, name = os.path.split(pathname)
name, ext = os.path.splitext(name) name, ext = os.path.splitext(name)
fp = open(pathname, READ_MODE) with open(pathname, READ_MODE) as fp:
stuff = (ext, "r", imp.PY_SOURCE) stuff = (ext, "r", imp.PY_SOURCE)
self.load_module(name, fp, pathname, stuff) self.load_module(name, fp, pathname, stuff)
...@@ -461,6 +461,8 @@ class ModuleFinder: ...@@ -461,6 +461,8 @@ class ModuleFinder:
fp, buf, stuff = self.find_module("__init__", m.__path__) fp, buf, stuff = self.find_module("__init__", m.__path__)
self.load_module(fqname, fp, buf, stuff) self.load_module(fqname, fp, buf, stuff)
self.msgout(2, "load_package ->", m) self.msgout(2, "load_package ->", m)
if fp:
fp.close()
return m return m
def add_module(self, fqname): def add_module(self, fqname):
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment