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 /main.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 'main.py')
-rwxr-xr-x | main.py | 10 |
1 files changed, 5 insertions, 5 deletions
@@ -88,7 +88,7 @@ class _Repo(object): | |||
88 | glob = argv | 88 | glob = argv |
89 | name = 'help' | 89 | name = 'help' |
90 | argv = [] | 90 | argv = [] |
91 | gopts, gargs = global_options.parse_args(glob) | 91 | gopts, _gargs = global_options.parse_args(glob) |
92 | 92 | ||
93 | if gopts.trace: | 93 | if gopts.trace: |
94 | SetTrace() | 94 | SetTrace() |
@@ -182,8 +182,8 @@ def _CheckWrapperVersion(ver, repo_path): | |||
182 | repo_path = '~/bin/repo' | 182 | repo_path = '~/bin/repo' |
183 | 183 | ||
184 | if not ver: | 184 | if not ver: |
185 | print >>sys.stderr, 'no --wrapper-version argument' | 185 | print >>sys.stderr, 'no --wrapper-version argument' |
186 | sys.exit(1) | 186 | sys.exit(1) |
187 | 187 | ||
188 | exp = _CurrentWrapperVersion() | 188 | exp = _CurrentWrapperVersion() |
189 | ver = tuple(map(lambda x: int(x), ver.split('.'))) | 189 | ver = tuple(map(lambda x: int(x), ver.split('.'))) |
@@ -211,8 +211,8 @@ def _CheckWrapperVersion(ver, repo_path): | |||
211 | 211 | ||
212 | def _CheckRepoDir(dir): | 212 | def _CheckRepoDir(dir): |
213 | if not dir: | 213 | if not dir: |
214 | print >>sys.stderr, 'no --repo-dir argument' | 214 | print >>sys.stderr, 'no --repo-dir argument' |
215 | sys.exit(1) | 215 | sys.exit(1) |
216 | 216 | ||
217 | def _PruneOptions(argv, opt): | 217 | def _PruneOptions(argv, opt): |
218 | i = 0 | 218 | i = 0 |