repair: define explicit local variable, don't reuse a comprehension variable
authorMike Edgar <adgar@google.com>
Tue, 10 Mar 2015 16:25:10 -0400
changeset 24252 f962692853c0
parent 24251 a330660b3d71
child 24253 26fa5ff9e660
repair: define explicit local variable, don't reuse a comprehension variable The node ID used in strip bundle names is currently taken as the last iterated value in a list comprehension found much earlier in the function. This change makes the node selection more explicit at the cost of redundancy.
mercurial/repair.py
--- a/mercurial/repair.py	Sat Feb 07 12:49:30 2015 -0800
+++ b/mercurial/repair.py	Tue Mar 10 16:25:10 2015 -0400
@@ -137,6 +137,7 @@
     # create a changegroup for all the branches we need to keep
     backupfile = None
     vfs = repo.vfs
+    node = nodelist[-1]
     if backup:
         backupfile = _bundle(repo, stripbases, cl.heads(), node, topic)
         repo.ui.status(_("saved backup bundle to %s\n") %