diff options
| author | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-10-25 12:23:11 +0900 |
|---|---|---|
| committer | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-10-30 10:28:20 +0900 |
| commit | 1d947b30342163b723c96db563967323535fef45 (patch) | |
| tree | b6b02d02df01792b356b9e50ceeaadcaf3a2e8c5 /subcmds/upload.py | |
| parent | 2d113f35460051823ea54d61c5c939565518f969 (diff) | |
| download | git-repo-1d947b30342163b723c96db563967323535fef45.tar.gz | |
Even more coding style cleanup
Fixing some more pylint warnings:
W1401: Anomalous backslash in string
W0623: Redefining name 'name' from outer scope
W0702: No exception type(s) specified
E0102: name: function already defined line n
Change-Id: I5afcdb4771ce210390a79981937806e30900a93c
Diffstat (limited to 'subcmds/upload.py')
| -rw-r--r-- | subcmds/upload.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/subcmds/upload.py b/subcmds/upload.py index 84a5e440..39721ac2 100644 --- a/subcmds/upload.py +++ b/subcmds/upload.py | |||
| @@ -297,7 +297,7 @@ Gerrit Code Review: http://code.google.com/p/gerrit/ | |||
| 297 | try: | 297 | try: |
| 298 | # refs/changes/XYZ/N --> XYZ | 298 | # refs/changes/XYZ/N --> XYZ |
| 299 | return refs.get(last_pub).split('/')[-2] | 299 | return refs.get(last_pub).split('/')[-2] |
| 300 | except: | 300 | except (AttributeError, IndexError): |
| 301 | return "" | 301 | return "" |
| 302 | 302 | ||
| 303 | def _UploadAndReport(self, opt, todo, original_people): | 303 | def _UploadAndReport(self, opt, todo, original_people): |