diff options
author | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-09-24 12:15:13 +0900 |
---|---|---|
committer | Gustaf Lundh <gustaf.lundh@sonymobile.com> | 2012-10-09 12:45:30 +0200 |
commit | 8a68ff96057ec58e524a3e41a2d8dca7b5d016bc (patch) | |
tree | 22f6971e8d3c4a90d11d3704602d073a852328b4 /subcmds/help.py | |
parent | e3b1c45aebed329cbc9ad172b1d8e812cf208117 (diff) | |
download | git-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/help.py')
-rw-r--r-- | subcmds/help.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/subcmds/help.py b/subcmds/help.py index 0df3c14b..375d04d2 100644 --- a/subcmds/help.py +++ b/subcmds/help.py | |||
@@ -120,8 +120,8 @@ See 'repo help --all' for a complete list of recognized commands. | |||
120 | m = asciidoc_hdr.match(para) | 120 | m = asciidoc_hdr.match(para) |
121 | if m: | 121 | if m: |
122 | title = m.group(1) | 122 | title = m.group(1) |
123 | type = m.group(2) | 123 | section_type = m.group(2) |
124 | if type[0] in ('=', '-'): | 124 | if section_type[0] in ('=', '-'): |
125 | p = self.heading | 125 | p = self.heading |
126 | else: | 126 | else: |
127 | def _p(fmt, *args): | 127 | def _p(fmt, *args): |
@@ -131,7 +131,7 @@ See 'repo help --all' for a complete list of recognized commands. | |||
131 | 131 | ||
132 | p('%s', title) | 132 | p('%s', title) |
133 | self.nl() | 133 | self.nl() |
134 | p('%s', ''.ljust(len(title),type[0])) | 134 | p('%s', ''.ljust(len(title),section_type[0])) |
135 | self.nl() | 135 | self.nl() |
136 | continue | 136 | continue |
137 | 137 | ||