[PATCH 3 of 8] hidden: change _domainancestors() to _revealancestors()

Pierre-Yves David pierre-yves.david at ens-lyon.org
Mon May 29 12:50:33 EDT 2017



On 05/28/2017 08:15 AM, Martin von Zweigbergk wrote:
> # HG changeset patch
> # User Martin von Zweigbergk <martinvonz at google.com>
> # Date 1495945026 25200
> #      Sat May 27 21:17:06 2017 -0700
> # Node ID aef3194855e763ecced80c2df6bd9672207aefbc
> # Parent  a0434758fd9a95ea3807ff4766eaedff6852c628
> hidden: change _domainancestors() to _revealancestors()
>
> This change makes the function will actually reveal the ancestors by
> removing them from the hidden set. This prepares for further
> simplification.

That change looks good to me but for a small feedback on the function 
signature.

> diff --git a/mercurial/repoview.py b/mercurial/repoview.py
> --- a/mercurial/repoview.py
> +++ b/mercurial/repoview.py
> @@ -59,9 +59,11 @@
>                  break
>      return blockers
>
> -def _domainancestors(pfunc, revs, domain):
> -    """return ancestors of 'revs' within 'domain'
> +def _revealancestors(hidden, pfunc, revs, domain):

small nit: other functions that use "pfunc" always takes it as first 
parameter. I would rather see "pfunc" remains the first parameters here too.

-- 
Pierre-Yves David


More information about the Mercurial-devel mailing list