commit 1d947b30342163b723c96db563967323535fef45
author David Pursehouse <david.pursehouse@sonymobile.com> Thu Oct 25 12:23:11 2012 +0900
committer David Pursehouse <david.pursehouse@sonymobile.com> Tue Oct 30 10:28:20 2012 +0900
tree b6b02d02df01792b356b9e50ceeaadcaf3a2e8c5
parent 2d113f35460051823ea54d61c5c939565518f969

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

subcmds/upload.py

diff --git a/subcmds/upload.py b/subcmds/upload.py
index 84a5e44..39721ac 100644
--- a/subcmds/upload.py
+++ b/subcmds/upload.py
@@ -297,7 +297,7 @@
     try:
       # refs/changes/XYZ/N --> XYZ
       return refs.get(last_pub).split('/')[-2]
-    except:
+    except (AttributeError, IndexError):
       return ""
 
   def _UploadAndReport(self, opt, todo, original_people):