summaryrefslogtreecommitdiffstats
path: root/subcmds/status.py
diff options
context:
space:
mode:
authorDavid Pursehouse <david.pursehouse@sonymobile.com>2012-09-24 12:15:13 +0900
committerGustaf Lundh <gustaf.lundh@sonymobile.com>2012-10-09 12:45:30 +0200
commit8a68ff96057ec58e524a3e41a2d8dca7b5d016bc (patch)
tree22f6971e8d3c4a90d11d3704602d073a852328b4 /subcmds/status.py
parente3b1c45aebed329cbc9ad172b1d8e812cf208117 (diff)
downloadgit-repo-8a68ff96057ec58e524a3e41a2d8dca7b5d016bc.tar.gz
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
Diffstat (limited to 'subcmds/status.py')
-rw-r--r--subcmds/status.py8
1 files changed, 4 insertions, 4 deletions
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:
98 sem.release() 98 sem.release()
99 99
100 def Execute(self, opt, args): 100 def Execute(self, opt, args):
101 all = self.GetProjects(args) 101 all_projects = self.GetProjects(args)
102 counter = itertools.count() 102 counter = itertools.count()
103 103
104 if opt.jobs == 1: 104 if opt.jobs == 1:
105 for project in all: 105 for project in all_projects:
106 state = project.PrintWorkTreeStatus() 106 state = project.PrintWorkTreeStatus()
107 if state == 'CLEAN': 107 if state == 'CLEAN':
108 counter.next() 108 counter.next()
109 else: 109 else:
110 sem = _threading.Semaphore(opt.jobs) 110 sem = _threading.Semaphore(opt.jobs)
111 threads_and_output = [] 111 threads_and_output = []
112 for project in all: 112 for project in all_projects:
113 sem.acquire() 113 sem.acquire()
114 114
115 class BufList(StringIO.StringIO): 115 class BufList(StringIO.StringIO):
@@ -128,5 +128,5 @@ the following meanings:
128 t.join() 128 t.join()
129 output.dump(sys.stdout) 129 output.dump(sys.stdout)
130 output.close() 130 output.close()
131 if len(all) == counter.next(): 131 if len(all_projects) == counter.next():
132 print 'nothing to commit (working directory clean)' 132 print 'nothing to commit (working directory clean)'