[PATCH 02 of 10] check-seclevel: use a context manager for file I/O

Bryan O'Sullivan bos at serpentine.com
Tue Jan 12 16:50:05 CST 2016


# HG changeset patch
# User Bryan O'Sullivan <bryano at fb.com>
# Date 1452637696 28800
#      Tue Jan 12 14:28:16 2016 -0800
# Node ID 9f45e2dd68b429b1c2f14074b2b49a48281e71eb
# Parent  6919e06dd7cdb16638aa013653a34a951ea2cd4f
check-seclevel: use a context manager for file I/O

diff --git a/doc/check-seclevel.py b/doc/check-seclevel.py
--- a/doc/check-seclevel.py
+++ b/doc/check-seclevel.py
@@ -107,11 +107,8 @@ def checkfile(ui, filename, initlevel):
         filename = 'stdin'
         doc = sys.stdin.read()
     else:
-        fp = open(filename)
-        try:
+        with open(filename) as fp:
             doc = fp.read()
-        finally:
-            fp.close()
 
     ui.note(('checking input from %s with initlevel %d\n') %
             (filename, initlevel))


More information about the Mercurial-devel mailing list