chiark / gitweb /
scanner: more descriptive variable names from os.walk
authorMarcus Hoffmann <bubuiic@aol.com>
Mon, 4 Sep 2017 14:11:27 +0000 (16:11 +0200)
committerMarcus Hoffmann <bubuiic@aol.com>
Mon, 11 Sep 2017 14:49:30 +0000 (16:49 +0200)
This uses the names from python documentation of os.walk().

fdroidserver/scanner.py

index 52020c89c04deb1ce4a57767375407414331f793..87b7ed630102d5e3db7523a762224c7917ff271d 100644 (file)
@@ -165,20 +165,20 @@ def scan_source(build_dir, build):
         return any(command.match(line) for command in gradle_compile_commands)
 
     # Iterate through all files in the source code
-    for r, d, f in os.walk(build_dir, topdown=True):
+    for dirpath, dirnames, filenames in os.walk(build_dir, topdown=True):
 
         # It's topdown, so checking the basename is enough
         for ignoredir in ('.hg', '.git', '.svn', '.bzr'):
-            if ignoredir in d:
-                d.remove(ignoredir)
+            if ignoredir in dirnames:
+                dirnames.remove(ignoredir)
 
-        for curfile in f:
+        for curfile in filenames:
 
             if curfile in ['.DS_Store']:
                 continue
 
             # Path (relative) to the file
-            fp = os.path.join(r, curfile)
+            fp = os.path.join(dirpath, curfile)
 
             if os.path.islink(fp):
                 continue