diff options
author | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-10-11 16:44:48 +0900 |
---|---|---|
committer | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-10-22 12:30:14 +0900 |
commit | 5c6eeac8f0350fd6b14cf226ffcff655f1dd9582 (patch) | |
tree | 3225695b9d2a97342a49127717ea5e2bc5935a63 /subcmds/checkout.py | |
parent | e98607248eec2b149d84efe944c12cbef419b82e (diff) | |
download | git-repo-5c6eeac8f0350fd6b14cf226ffcff655f1dd9582.tar.gz |
More coding style cleanup
Fixing more issues found with pylint. Some that were supposed to
have been fixed in the previous sweep (Ie0db839e) but were missed:
C0321: More than one statement on a single line
W0622: Redefining built-in 'name'
And some more:
W0631: Using possibly undefined loop variable 'name'
W0223: Method 'name' is abstract in class 'name' but is not overridden
W0231: __init__ method from base class 'name' is not called
Change-Id: Ie119183708609d6279e973057a385fde864230c3
Diffstat (limited to 'subcmds/checkout.py')
-rw-r--r-- | subcmds/checkout.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/subcmds/checkout.py b/subcmds/checkout.py index 533d20e1..bfbe9921 100644 --- a/subcmds/checkout.py +++ b/subcmds/checkout.py | |||
@@ -39,10 +39,10 @@ The command is equivalent to: | |||
39 | nb = args[0] | 39 | nb = args[0] |
40 | err = [] | 40 | err = [] |
41 | success = [] | 41 | success = [] |
42 | all = self.GetProjects(args[1:]) | 42 | all_projects = self.GetProjects(args[1:]) |
43 | 43 | ||
44 | pm = Progress('Checkout %s' % nb, len(all)) | 44 | pm = Progress('Checkout %s' % nb, len(all_projects)) |
45 | for project in all: | 45 | for project in all_projects: |
46 | pm.update() | 46 | pm.update() |
47 | 47 | ||
48 | status = project.CheckoutBranch(nb) | 48 | status = project.CheckoutBranch(nb) |