From 8a68ff96057ec58e524a3e41a2d8dca7b5d016bc Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Mon, 24 Sep 2012 12:15:13 +0900 Subject: 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 --- subcmds/status.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'subcmds/status.py') diff --git a/subcmds/status.py b/subcmds/status.py index 75d68ebc..7611621e 100644 --- a/subcmds/status.py +++ b/subcmds/status.py @@ -98,18 +98,18 @@ the following meanings: 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 @@ the following meanings: 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)' -- cgit v1.2.3-54-g00ecf