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 /error.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 'error.py')
-rw-r--r-- | error.py | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -85,8 +85,8 @@ class RepoChangedException(Exception): | |||
85 | repo or manifest repositories. In this special case we must | 85 | repo or manifest repositories. In this special case we must |
86 | use exec to re-execute repo with the new code and manifest. | 86 | use exec to re-execute repo with the new code and manifest. |
87 | """ | 87 | """ |
88 | def __init__(self, extra_args=[]): | 88 | def __init__(self, extra_args=None): |
89 | self.extra_args = extra_args | 89 | self.extra_args = extra_args or [] |
90 | 90 | ||
91 | class HookError(Exception): | 91 | class HookError(Exception): |
92 | """Thrown if a 'repo-hook' could not be run. | 92 | """Thrown if a 'repo-hook' could not be run. |