<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On May 29, 2017 9:50 AM, "Pierre-Yves David" <<a href="mailto:pierre-yves.david@ens-lyon.org">pierre-yves.david@ens-lyon.org</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="quoted-text"><br>
<br>
On 05/28/2017 08:15 AM, Martin von Zweigbergk wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
# HG changeset patch<br>
# User Martin von Zweigbergk <<a href="mailto:martinvonz@google.com" target="_blank">martinvonz@google.com</a>><br>
# Date 1495945026 25200<br>
#      Sat May 27 21:17:06 2017 -0700<br>
# Node ID aef3194855e763ecced80c2df6bd96<wbr>72207aefbc<br>
# Parent  a0434758fd9a95ea3807ff4766eaed<wbr>ff6852c628<br>
hidden: change _domainancestors() to _revealancestors()<br>
<br>
This change makes the function will actually reveal the ancestors by<br>
removing them from the hidden set. This prepares for further<br>
simplification.<br>
</blockquote>
<br></div>
That change looks good to me but for a small feedback on the function signature.<div class="quoted-text"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
diff --git a/mercurial/repoview.py b/mercurial/repoview.py<br>
--- a/mercurial/repoview.py<br>
+++ b/mercurial/repoview.py<br>
@@ -59,9 +59,11 @@<br>
                 break<br>
     return blockers<br>
<br>
-def _domainancestors(pfunc, revs, domain):<br>
-    """return ancestors of 'revs' within 'domain'<br>
+def _revealancestors(hidden, pfunc, revs, domain):<br>
</blockquote>
<br></div>
small nit: other functions that use "pfunc" always takes it as first parameter. I would rather see "pfunc" remains the first parameters here too.</blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">'hidden' gets mutated, and mutated parameters are usually first, I think (including, but probably not limited to, 'self'). Don't you think that should trump the pfunc-first convention?</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><font color="#888888"><br>
<br>
-- <br>
Pierre-Yves David<br>
</font></blockquote></div><br></div></div></div>