diff Tools/scripts/patchcheck.py @ 71667:d72e04ff0f97

Small cleanup
author Éric Araujo <merwok@netwok.org>
date Sun, 31 Jul 2011 18:41:25 +0200
parents 5993f91598ce
children d403eaec64df
line wrap: on
line diff
--- a/Tools/scripts/patchcheck.py	Sun Jul 31 18:33:00 2011 +0200
+++ b/Tools/scripts/patchcheck.py	Sun Jul 31 18:41:25 2011 +0200
@@ -39,18 +39,13 @@
 @status("Getting the list of files that have been added/changed",
         info=lambda x: n_files_str(len(x)))
 def changed_files():
-    """Get the list of changed or added files from the VCS."""
-    if os.path.isdir(os.path.join(SRCDIR, '.hg')):
-        cmd = 'hg status --added --modified --no-status'
-    else:
+    """Get the list of changed or added files from Mercurial."""
+    if not os.path.isdir(os.path.join(SRCDIR, '.hg')):
         sys.exit('need a checkout to get modified files')
 
-    st = subprocess.Popen(cmd.split(), stdout=subprocess.PIPE)
-    try:
-        st.wait()
+    cmd = 'hg status --added --modified --no-status'
+    with subprocess.Popen(cmd.split(), stdout=subprocess.PIPE) as st:
         return [x.decode().rstrip() for x in st.stdout]
-    finally:
-        st.stdout.close()
 
 
 def report_modified_files(file_paths):