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/prune.py b/subcmds/prune.py
index f412bd4..c50a550 100644
--- a/subcmds/prune.py
+++ b/subcmds/prune.py
@@ -24,11 +24,11 @@
"""
def Execute(self, opt, args):
- all = []
+ all_branches = []
for project in self.GetProjects(args):
- all.extend(project.PruneHeads())
+ all_branches.extend(project.PruneHeads())
- if not all:
+ if not all_branches:
return
class Report(Coloring):
@@ -36,13 +36,13 @@
Coloring.__init__(self, config, 'status')
self.project = self.printer('header', attr='bold')
- out = Report(all[0].project.config)
+ out = Report(all_branches[0].project.config)
out.project('Pending Branches')
out.nl()
project = None
- for branch in all:
+ for branch in all_branches:
if project != branch.project:
project = branch.project
out.nl()