chiark / gitweb /
lint: fix duplicate usage of appid
authorDaniel Martí <mvdan@mvdan.cc>
Tue, 26 Aug 2014 11:31:31 +0000 (13:31 +0200)
committerDaniel Martí <mvdan@mvdan.cc>
Tue, 26 Aug 2014 11:33:17 +0000 (13:33 +0200)
fdroidserver/lint.py

index 30a1807c57b0852cd6a58a5e104e04fff983c6b6..9c0dcfdc23eb26320656d42dcd658e93a74be425 100644 (file)
@@ -128,16 +128,16 @@ regex_pedantic = {
 
 def main():
 
-    global config, options, appid, count
-    appid = None
+    global config, options, curid, count
+    curid = None
 
     count = Counter()
 
     def warn(message):
-        global appid, count
-        if appid:
-            print "%s:" % appid
-            appid = None
+        global curid, count
+        if curid:
+            print "%s:" % curid
+            curid = None
             count['app'] += 1
         print '    %s' % message
         count['warn'] += 1
@@ -163,6 +163,7 @@ def main():
     apps = common.read_app_args(args, allapps, False)
 
     for appid, app in apps.iteritems():
+        curid = appid
         lastcommit = ''
 
         if app['Disabled']:
@@ -240,7 +241,7 @@ def main():
                         # TODO: This should not be pedantic!
                         pwarn("'%s' used in %s '%s'" % (s, flag, build[flag]))
 
-        if not appid:
+        if not curid:
             print
 
     logging.info("Found a total of %i warnings in %i apps." % (count['warn'], count['app']))