changeset 10716:5f92bde72eef

check-code: Only call check-code if __name__ = "__main__". This changeset moves the code that actually executes something to the "if __name__ = '__main__'" section to allow the file to be imported as a module.
author Pierre-Yves David <pierre-yves.david@logilab.fr>
date Tue, 16 Mar 2010 19:52:56 +0100
parents 83dce0f817f4
children b1f4fcef99b3
files contrib/check-code.py
diffstat 1 files changed, 32 insertions(+), 31 deletions(-) [+]
line wrap: on
line diff
--- a/contrib/check-code.py	Tue Mar 16 19:52:55 2010 +0100
+++ b/contrib/check-code.py	Tue Mar 16 19:52:56 2010 +0100
@@ -134,36 +134,37 @@
     ('c', r'.*\.c$', cfilters, cpats),
 ]
 
-if len(sys.argv) == 1:
-    check = glob.glob("*")
-else:
-    check = sys.argv[1:]
+if __name__ == "__main__":
+    if len(sys.argv) == 1:
+        check = glob.glob("*")
+    else:
+        check = sys.argv[1:]
 
-for f in check:
-    for name, match, filters, pats in checks:
-        fc = 0
-        if not re.match(match, f):
-            continue
-        pre = post = open(f).read()
-        if "no-" + "check-code" in pre:
+    for f in check:
+        for name, match, filters, pats in checks:
+            fc = 0
+            if not re.match(match, f):
+                continue
+            pre = post = open(f).read()
+            if "no-" + "check-code" in pre:
+                break
+            for p, r in filters:
+                post = re.sub(p, r, post)
+            # print post # uncomment to show filtered version
+            z = enumerate(zip(pre.splitlines(), post.splitlines(True)))
+            for n, l in z:
+                if "check-code" + "-ignore" in l[0]:
+                    continue
+                lc = 0
+                for p, msg in pats:
+                    if re.search(p, l[1]):
+                        if not lc:
+                            print "%s:%d:" % (f, n + 1)
+                            print " > %s" % l[0]
+                        print " %s" % msg
+                        lc += 1
+                        fc += 1
+                if fc == 15:
+                    print " (too many errors, giving up)"
+                    break
             break
-        for p, r in filters:
-            post = re.sub(p, r, post)
-        # print post # uncomment to show filtered version
-        z = enumerate(zip(pre.splitlines(), post.splitlines(True)))
-        for n, l in z:
-            if "check-code" + "-ignore" in l[0]:
-                continue
-            lc = 0
-            for p, msg in pats:
-                if re.search(p, l[1]):
-                    if not lc:
-                        print "%s:%d:" % (f, n + 1)
-                        print " > %s" % l[0]
-                    print " %s" % msg
-                    lc += 1
-                    fc += 1
-            if fc == 15:
-                print " (too many errors, giving up)"
-                break
-        break