Mercurial > evolve
changeset 3558:cec6e21b3bcf
evolve: de-duplicate some function calls
_cleanup() was called in continueevolve() and evolve() function too. Let's call
it at a single place for both the cases. Also call progresscb() outside of
conditionals to call it both times.
author | Pulkit Goyal <7895pulkit@gmail.com> |
---|---|
date | Fri, 16 Mar 2018 17:00:01 +0530 |
parents | 77a400149758 |
children | d166b3dc5e8a |
files | hgext3rd/evolve/evolvecmd.py |
diffstat | 1 files changed, 6 insertions(+), 5 deletions(-) [+] |
line wrap: on
line diff
--- a/hgext3rd/evolve/evolvecmd.py Fri Mar 16 16:11:22 2018 +0530 +++ b/hgext3rd/evolve/evolvecmd.py Fri Mar 16 17:00:01 2018 +0530 @@ -1130,6 +1130,10 @@ raise error.Abort('no evolve to continue') evolvestate.load() continueevolve(ui, repo, evolvestate, progresscb) + if evolvestate['command'] != 'evolve': + evolvestate.delete() + return + startnode = repo.unfiltered()[evolvestate['startnode']] evolvestate.delete() else: cmdutil.bailifchanged(repo) @@ -1160,8 +1164,8 @@ evolvestate['replacements'][curctx.node()] = [ret[1]] else: evolvestate['skippedrevs'].append(curctx.node()) - progresscb() - _cleanup(ui, repo, startnode, showprogress) + progresscb() + _cleanup(ui, repo, startnode, showprogress) def continueevolve(ui, repo, evolvestate, progresscb): """logic for handling of `hg evolve --continue`""" @@ -1227,7 +1231,6 @@ evolvestate['replacements'][ctx.node()] = node category = evolvestate['category'] confirm = evolvestate['confirm'] - startnode = evolvestate['startnode'] unfi = repo.unfiltered() for rev in evolvestate['revs']: # XXX: prevent this lookup by storing nodes instead of revnums @@ -1240,6 +1243,4 @@ evolvestate['replacements'][curctx.node()] = newnode[1] else: evolvestate['skippedrevs'].append(curctx.node()) - - _cleanup(ui, repo, unfi[startnode], True) return