From: Karl Hasselström <kha@treskal.com>
Date: Mon, 28 Jan 2008 21:20:09 +0000 (+0100)
Subject: Eliminate temp variable that's used just once
X-Git-Tag: v0.15-rc1~293
X-Git-Url: https://git.distorted.org.uk/~mdw/stgit/commitdiff_plain/9ee0285b485ac09a9407c4046107486cdb16c8c0?ds=inline

Eliminate temp variable that's used just once

Signed-off-by: Karl Hasselström <kha@treskal.com>
---

diff --git a/stgit/lib/transaction.py b/stgit/lib/transaction.py
index 809eabf..6a2ed81 100644
--- a/stgit/lib/transaction.py
+++ b/stgit/lib/transaction.py
@@ -174,7 +174,6 @@ class StackTransaction(object):
         """Attempt to push the named patch. If this results in conflicts,
         halts the transaction. If index+worktree are given, spill any
         conflicts to them."""
-        i = self.unapplied.index(pn)
         cd = self.patches[pn].data
         cd = cd.set_committer(None)
         s = ['', ' (empty)'][cd.is_nochange()]
@@ -203,7 +202,7 @@ class StackTransaction(object):
                 s = ' (conflict)'
         cd = cd.set_tree(tree)
         self.patches[pn] = self.__stack.repository.commit(cd)
-        del self.unapplied[i]
+        del self.unapplied[self.unapplied.index(pn)]
         self.applied.append(pn)
         out.info('Pushed %s%s' % (pn, s))
         if merge_conflict: