diff options
| -rwxr-xr-x | Lib/base64.py | 3 | ||||
| -rw-r--r-- | Lib/mimetypes.py | 7 | ||||
| -rw-r--r-- | Lib/sysconfig.py | 3 | ||||
| -rw-r--r-- | Lib/test/test_argparse.py | 3 | ||||
| -rw-r--r-- | Lib/test/test_xml_etree.py | 23 |
5 files changed, 25 insertions, 14 deletions
diff --git a/Lib/base64.py b/Lib/base64.py index 41a5e14..85204dd 100755 --- a/Lib/base64.py +++ b/Lib/base64.py @@ -343,7 +343,8 @@ def test(): if o == '-u': func = decode if o == '-t': test1(); return if args and args[0] != '-': - func(open(args[0], 'rb'), sys.stdout) + with open(args[0], 'rb') as f: + func(f, sys.stdout) else: func(sys.stdin, sys.stdout) diff --git a/Lib/mimetypes.py b/Lib/mimetypes.py index e6f72ae..816f6cd 100644 --- a/Lib/mimetypes.py +++ b/Lib/mimetypes.py @@ -199,9 +199,8 @@ class MimeTypes: list of standard types, else to the list of non-standard types. """ - fp = open(filename) - self.readfp(fp, strict) - fp.close() + with open(filename) as fp: + self.readfp(fp, strict) def readfp(self, fp, strict=True): """ @@ -356,7 +355,7 @@ def init(files=None): files = knownfiles for file in files: if os.path.isfile(file): - db.readfp(open(file)) + db.read(file) encodings_map = db.encodings_map suffix_map = db.suffix_map types_map = db.types_map[True] diff --git a/Lib/sysconfig.py b/Lib/sysconfig.py index 5d5d5c0..c40bb03 100644 --- a/Lib/sysconfig.py +++ b/Lib/sysconfig.py @@ -289,7 +289,8 @@ def _init_posix(vars): # load the installed pyconfig.h: config_h = get_config_h_filename() try: - parse_config_h(open(config_h), vars) + with open(config_h) as f: + parse_config_h(f, vars) except IOError, e: msg = "invalid Python installation: unable to open %s" % config_h if hasattr(e, "strerror"): diff --git a/Lib/test/test_argparse.py b/Lib/test/test_argparse.py index 3e1bf25..a96bb5c 100644 --- a/Lib/test/test_argparse.py +++ b/Lib/test/test_argparse.py @@ -4171,7 +4171,8 @@ class TestEncoding(TestCase): def _test_module_encoding(self, path): path, _ = os.path.splitext(path) path += ".py" - codecs.open(path, 'r', 'utf8').read() + with codecs.open(path, 'r', 'utf8') as f: + f.read() def test_argparse_module_encoding(self): self._test_module_encoding(argparse.__file__) diff --git a/Lib/test/test_xml_etree.py b/Lib/test/test_xml_etree.py index 98ca4d2..90afdfd 100644 --- a/Lib/test/test_xml_etree.py +++ b/Lib/test/test_xml_etree.py @@ -586,10 +586,13 @@ def parsefile(): <ns0:empty-element /> </ns0:root> + >>> with open(SIMPLE_XMLFILE) as f: + ... data = f.read() + >>> parser = ET.XMLParser() >>> parser.version # doctest: +ELLIPSIS 'Expat ...' - >>> parser.feed(open(SIMPLE_XMLFILE).read()) + >>> parser.feed(data) >>> print serialize(parser.close()) <root> <element key="value">text</element> @@ -598,7 +601,7 @@ def parsefile(): </root> >>> parser = ET.XMLTreeBuilder() # 1.2 compatibility - >>> parser.feed(open(SIMPLE_XMLFILE).read()) + >>> parser.feed(data) >>> print serialize(parser.close()) <root> <element key="value">text</element> @@ -608,7 +611,7 @@ def parsefile(): >>> target = ET.TreeBuilder() >>> parser = ET.XMLParser(target=target) - >>> parser.feed(open(SIMPLE_XMLFILE).read()) + >>> parser.feed(data) >>> print serialize(parser.close()) <root> <element key="value">text</element> @@ -711,7 +714,8 @@ def iterparse(): end-ns None >>> events = ("start", "end", "bogus") - >>> context = iterparse(SIMPLE_XMLFILE, events) + >>> with open(SIMPLE_XMLFILE, "rb") as f: + ... iterparse(f, events) Traceback (most recent call last): ValueError: unknown event 'bogus' @@ -763,6 +767,8 @@ def custom_builder(): """ Test parser w. custom builder. + >>> with open(SIMPLE_XMLFILE) as f: + ... data = f.read() >>> class Builder: ... def start(self, tag, attrib): ... print "start", tag @@ -772,7 +778,7 @@ def custom_builder(): ... pass >>> builder = Builder() >>> parser = ET.XMLParser(target=builder) - >>> parser.feed(open(SIMPLE_XMLFILE, "r").read()) + >>> parser.feed(data) start root start element end element @@ -782,6 +788,8 @@ def custom_builder(): end empty-element end root + >>> with open(SIMPLE_NS_XMLFILE) as f: + ... data = f.read() >>> class Builder: ... def start(self, tag, attrib): ... print "start", tag @@ -795,7 +803,7 @@ def custom_builder(): ... print "comment", repr(data) >>> builder = Builder() >>> parser = ET.XMLParser(target=builder) - >>> parser.feed(open(SIMPLE_NS_XMLFILE, "r").read()) + >>> parser.feed(data) pi pi 'data' comment ' comment ' start {namespace}root @@ -813,7 +821,8 @@ def getchildren(): """ Test Element.getchildren() - >>> tree = ET.parse(open(SIMPLE_XMLFILE, "r")) + >>> with open(SIMPLE_XMLFILE, "r") as f: + ... tree = ET.parse(f) >>> for elem in tree.getroot().iter(): ... summarize_list(elem.getchildren()) ['element', 'element', 'empty-element'] |