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/branches.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'subcmds/branches.py') diff --git a/subcmds/branches.py b/subcmds/branches.py index a4f8d360..81aa5b18 100644 --- a/subcmds/branches.py +++ b/subcmds/branches.py @@ -140,12 +140,12 @@ is shown, then the branch appears in all projects. fmt = out.write paths = [] if in_cnt < project_cnt - in_cnt: - type = 'in' + in_type = 'in' for b in i.projects: paths.append(b.project.relpath) else: fmt = out.notinproject - type = 'not in' + in_type = 'not in' have = set() for b in i.projects: have.add(b.project) @@ -153,11 +153,11 @@ is shown, then the branch appears in all projects. if not p in have: paths.append(p.relpath) - s = ' %s %s' % (type, ', '.join(paths)) + s = ' %s %s' % (in_type, ', '.join(paths)) if width + 7 + len(s) < 80: fmt(s) else: - fmt(' %s:' % type) + fmt(' %s:' % in_type) for p in paths: out.nl() fmt(width*' ' + ' %s' % p) -- cgit v1.2.3-54-g00ecf