ignore: refactor file read into a function
authorDurham Goode <durham@fb.com>
Thu, 07 May 2015 20:57:37 -0700
changeset 25065 8cf7f0c4cb14
parent 25064 3bbbadf69d0a
child 25066 e91b32d3c67b
ignore: refactor file read into a function This refactors the ignore file reading code into a function so that in a future patch we can make it recursive.
mercurial/ignore.py
--- a/mercurial/ignore.py	Thu May 14 01:49:10 2015 +0900
+++ b/mercurial/ignore.py	Thu May 07 20:57:37 2015 -0700
@@ -52,6 +52,20 @@
 
     return patterns, warnings
 
+def readignorefile(filepath, warn, skipwarning=False):
+    try:
+        pats = []
+        fp = open(filepath)
+        pats, warnings = ignorepats(fp)
+        fp.close()
+        for warning in warnings:
+            warn("%s: %s\n" % (filepath, warning))
+    except IOError, inst:
+        if not skipwarning:
+            warn(_("skipping unreadable ignore file '%s': %s\n") %
+                 (filepath, inst.strerror))
+    return pats
+
 def readpats(root, files, warn):
     '''return a dict mapping ignore-file-name to list-of-patterns'''
 
@@ -59,17 +73,9 @@
     for f in files:
         if f in pats:
             continue
-        try:
-            pats[f] = []
-            fp = open(f)
-            pats[f], warnings = ignorepats(fp)
-            fp.close()
-            for warning in warnings:
-                warn("%s: %s\n" % (f, warning))
-        except IOError, inst:
-            if f != files[0]:
-                warn(_("skipping unreadable ignore file '%s': %s\n") %
-                     (f, inst.strerror))
+        skipwarning = f == files[0]
+        pats[f] = readignorefile(f, warn, skipwarning=skipwarning)
+
     return [(f, pats[f]) for f in files if f in pats]
 
 def ignore(root, files, warn):