Mercurial Mercurial > pub > hg > evolve / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | zip | gz | bz2 | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
simplify locking in prune stable
authorPierre-Yves David <pierre-yves.david@ens-lyon.org>
Mon, 04 Feb 2013 17:55:03 +0100
branchstable
changeset 683 6d24bd455f79
parent 682 6cbb52a8401b
child 684 19a1a8f93c07
simplify locking in prune
hgext/evolve.py file | annotate | diff | comparison | revisions 
--- a/hgext/evolve.py	Mon Feb 04 17:51:27 2013 +0100
+++ b/hgext/evolve.py	Mon Feb 04 17:55:03 2013 +0100
@@ -1255,35 +1255,33 @@
     revs = list(revs)
     revs.extend(opts['rev'])
     succs = opts['new'] + opts['succ']
+    wlock = lock = None
     wlock = repo.wlock()
     try:
         lock = repo.lock()
-        try:
-            new = set(scmutil.revrange(repo, succs))
-            targetnodes = set(scmutil.revrange(repo, revs))
-            if not targetnodes:
-                raise util.Abort('nothing to prune')
-            if new:
-                sucs = tuple(repo[n] for n in new)
-            else:
-                sucs = ()
-            markers = []
-            for n in targetnodes:
-                markers.append((repo[n], sucs))
-            createmarkers(repo, markers)
+        new = set(scmutil.revrange(repo, succs))
+        targetnodes = set(scmutil.revrange(repo, revs))
+        if not targetnodes:
+            raise util.Abort('nothing to prune')
+        if new:
+            sucs = tuple(repo[n] for n in new)
+        else:
+            sucs = ()
+        markers = []
+        for n in targetnodes:
+            markers.append((repo[n], sucs))
+        createmarkers(repo, markers)
 
-            # update to an unkilled parent
-            wdp = repo['.']
-            newnode = wdp
-            while newnode.obsolete():
-                newnode = newnode.parents()[0]
-            if newnode.node() != wdp.node():
-                commands.update(ui, repo, newnode.rev())
-                ui.status(_('working directory now at %s\n') % newnode)
-        finally:
-            lock.release()
+        # update to an unkilled parent
+        wdp = repo['.']
+        newnode = wdp
+        while newnode.obsolete():
+            newnode = newnode.parents()[0]
+        if newnode.node() != wdp.node():
+            commands.update(ui, repo, newnode.rev())
+            ui.status(_('working directory now at %s\n') % newnode)
     finally:
-        wlock.release()
+        lockmod.release(lock, wlock)
 
 @command('amend|refresh',
     [('A', 'addremove', None,
evolve