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/start.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/start.py')
| -rw-r--r-- | subcmds/start.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/subcmds/start.py b/subcmds/start.py index 00885076..be645314 100644 --- a/subcmds/start.py +++ b/subcmds/start.py | |||
| @@ -52,10 +52,10 @@ revision specified in the manifest. | |||
| 52 | print >>sys.stderr, "error: at least one project must be specified" | 52 | print >>sys.stderr, "error: at least one project must be specified" |
| 53 | sys.exit(1) | 53 | sys.exit(1) |
| 54 | 54 | ||
| 55 | all = self.GetProjects(projects) | 55 | all_projects = self.GetProjects(projects) |
| 56 | 56 | ||
| 57 | pm = Progress('Starting %s' % nb, len(all)) | 57 | pm = Progress('Starting %s' % nb, len(all_projects)) |
| 58 | for project in all: | 58 | for project in all_projects: |
| 59 | pm.update() | 59 | pm.update() |
| 60 | # If the current revision is a specific SHA1 then we can't push back | 60 | # If the current revision is a specific SHA1 then we can't push back |
| 61 | # to it so substitute the manifest default revision instead. | 61 | # to it so substitute the manifest default revision instead. |