[PATCH 7 of 7] checkheads: simplify the code around obsolescence post-processing
Yuya Nishihara
yuya at tcha.org
Sun Jun 4 10:14:40 EDT 2017
On Sat, 03 Jun 2017 10:06:30 +0200, Pierre-Yves David wrote:
> # HG changeset patch
> # User Pierre-Yves David <pierre-yves.david at octobus.net>
> # Date 1496028210 -7200
> # Mon May 29 05:23:30 2017 +0200
> # Node ID 54ef0920630de5f0c3c666dbb30a73b1adcb91ec
> # Parent e3c199932c9c84037a05ffe817838a5a4bd6f5a8
> # EXP-Topic pushrace
> # Available At https://www.mercurial-scm.org/repo/users/marmoute/mercurial/
> # hg pull https://www.mercurial-scm.org/repo/users/marmoute/mercurial/ -r 54ef0920630d
> checkheads: simplify the code around obsolescence post-processing
Looks good to me. Queued these, thanks.
> diff --git a/mercurial/discovery.py b/mercurial/discovery.py
> --- a/mercurial/discovery.py
> +++ b/mercurial/discovery.py
> @@ -347,13 +347,10 @@ def checkheads(pushop):
> oldhs = set(remoteheads)
> oldhs.update(unsyncedheads)
> dhs = None # delta heads, the new heads on branch
> - if not repo.obsstore:
> - discardedheads = set()
> - newhs = set(newheads)
> - else:
> - newhs, discardedheads = _postprocessobsolete(pushop,
> - allfuturecommon,
> - newheads)
> + if repo.obsstore:
> + result = _postprocessobsolete(pushop, allfuturecommon, newheads)
> + newheads = sorted(result[0])
sorted() result is unused now, but maybe it would be necessary in future patch?
More information about the Mercurial-devel
mailing list