From: Victor Stinner <victor.stinner@haypocalc.com>
Date: Sun, 16 May 2010 00:36:38 +0000 (+0000)
Subject: Merged revisions 81224 via svnmerge from
X-Git-Tag: v3.2a1~791
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7ac5cb18df3b9778d55aaf673da7c49f1666b4c9;p=python

Merged revisions 81224 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r81224 | victor.stinner | 2010-05-16 02:34:40 +0200 (dim., 16 mai 2010) | 4 lines

  Use with open() as fo: ... instead of try: fo = open(...) finally: fo.close()

  fo is not set if the open() fails.
........
---

diff --git a/Lib/test/list_tests.py b/Lib/test/list_tests.py
index e2889cca6d..12f4e8f650 100644
--- a/Lib/test/list_tests.py
+++ b/Lib/test/list_tests.py
@@ -66,13 +66,11 @@ class CommonTest(seq_tests.CommonTest):
         d.append(d)
         d.append(400)
         try:
-            fo = open(support.TESTFN, "w")
-            fo.write(str(d))
-            fo.close()
-            fo = open(support.TESTFN, "r")
-            self.assertEqual(fo.read(), repr(d))
+            with open(support.TESTFN, "w") as fo:
+                fo.write(str(d))
+            with open(support.TESTFN, "r") as fo:
+                self.assertEqual(fo.read(), repr(d))
         finally:
-            fo.close()
             os.remove(support.TESTFN)
 
     def test_set_subscript(self):