diff options
author | Mike Frysinger <vapier@google.com> | 2019-06-13 02:13:23 -0400 |
---|---|---|
committer | Mike Frysinger <vapier@google.com> | 2019-06-13 13:39:25 +0000 |
commit | 31067c0ac583af2304ffe52c11df8b14c6162502 (patch) | |
tree | de1841787de87b13af60d9dd470d2544440c32d4 /subcmds/abandon.py | |
parent | 35159abbebf5453d4c9e89e570b891fb3d6c93cc (diff) | |
download | git-repo-31067c0ac583af2304ffe52c11df8b14c6162502.tar.gz |
tweak raise/dict syntax for Python 3 compat
Use the `raise` statement directly.
Switch to using .items() instead of .iteritems(). Python 3 doesn't
have .iteritems() as .items() is a generator, and these are small
enough that the Python 2 overhead should be negligible.
We have to run .keys() through list() in a few places as Python 3
uses a generator and we sometimes want to iterate more than once.
That's why we don't change all .keys() or .items() calls -- most
are in places where generators are fine.
Bug: https://crbug.com/gerrit/10418
Change-Id: I469899d9b77ffd77ccabb831bc4b217407fefe6f
Diffstat (limited to 'subcmds/abandon.py')
-rw-r--r-- | subcmds/abandon.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/subcmds/abandon.py b/subcmds/abandon.py index be32dc5c..f1c2a839 100644 --- a/subcmds/abandon.py +++ b/subcmds/abandon.py | |||
@@ -58,7 +58,7 @@ It is equivalent to "git branch -D <branchname>". | |||
58 | pm.update() | 58 | pm.update() |
59 | 59 | ||
60 | if opt.all: | 60 | if opt.all: |
61 | branches = project.GetBranches().keys() | 61 | branches = list(project.GetBranches().keys()) |
62 | else: | 62 | else: |
63 | branches = [nb] | 63 | branches = [nb] |
64 | 64 | ||