Coding style cleanup
Fix the following issues reported by pylint:
C0321: More than one statement on a single line
W0622: Redefining built-in 'name'
W0612: Unused variable 'name'
W0613: Unused argument 'name'
W0102: Dangerous default value 'value' as argument
W0105: String statement has no effect
Also fixed a few cases of inconsistent indentation.
Change-Id: Ie0db839e7c57d576cff12d8c055fe87030d00744
diff --git a/subcmds/status.py b/subcmds/status.py
index 75d68eb..7611621 100644
--- a/subcmds/status.py
+++ b/subcmds/status.py
@@ -98,18 +98,18 @@
sem.release()
def Execute(self, opt, args):
- all = self.GetProjects(args)
+ all_projects = self.GetProjects(args)
counter = itertools.count()
if opt.jobs == 1:
- for project in all:
+ for project in all_projects:
state = project.PrintWorkTreeStatus()
if state == 'CLEAN':
counter.next()
else:
sem = _threading.Semaphore(opt.jobs)
threads_and_output = []
- for project in all:
+ for project in all_projects:
sem.acquire()
class BufList(StringIO.StringIO):
@@ -128,5 +128,5 @@
t.join()
output.dump(sys.stdout)
output.close()
- if len(all) == counter.next():
+ if len(all_projects) == counter.next():
print 'nothing to commit (working directory clean)'