[PATCH] histedit: remove extra histedit constructor call
Ryan McElroy
rm at fb.com
Wed Mar 11 10:44:09 CDT 2015
On 3/11/2015 7:20 AM, Durham Goode wrote:
> # HG changeset patch
> # User Durham Goode <durham at fb.com>
> # Date 1426083595 25200
> # Wed Mar 11 07:19:55 2015 -0700
> # Node ID c00fe123cfeb0a8935a3b636da31f23eee6c67ba
> # Parent 913347bcd59c33110f029d83203b3b4c2ecd91a8
> histedit: remove extra histedit constructor call
>
> In a previous commit we removed the extra histedit object instance being
> constructed in --continue and --abort. The new --edit-todo missed this fix
> though (which means the state object it produces doesn't have the locks on it).
> It's not breaking anything now, but let's go ahead and clean that up before we
> forget.
>
> diff --git a/hgext/histedit.py b/hgext/histedit.py
> --- a/hgext/histedit.py
> +++ b/hgext/histedit.py
> @@ -616,7 +616,6 @@ def _histedit(ui, repo, state, *freeargs
> state.read()
> state = bootstrapcontinue(ui, state, opts)
> elif goal == 'edit-plan':
> - state = histeditstate(repo)
> state.read()
> if not rules:
> comment = editcomment % (state.parentctx, node.short(state.topmost))
Good catch. Looks good to me.
More information about the Mercurial-devel
mailing list