D4739: revset: remove part of the commonancestors() comment

valentin.gatienbaron (Valentin Gatien-Baron) phabricator at mercurial-scm.org
Wed Sep 26 07:35:00 EDT 2018


This revision was automatically updated to reflect the committed changes.
Closed by commit rHG3bc4fce0167f: revset: remove part of the commonancestors() comment (authored by valentin.gatienbaron, committed by ).

CHANGED PRIOR TO COMMIT
  https://phab.mercurial-scm.org/D4739?vs=11356&id=11365#toc

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D4739?vs=11356&id=11365

REVISION DETAIL
  https://phab.mercurial-scm.org/D4739

AFFECTED FILES
  mercurial/revset.py

CHANGE DETAILS

diff --git a/mercurial/revset.py b/mercurial/revset.py
--- a/mercurial/revset.py
+++ b/mercurial/revset.py
@@ -624,12 +624,6 @@
 @predicate('commonancestors(set)', safe=True)
 def commonancestors(repo, subset, x):
     """Returns all common ancestors of the set.
-
-    This method is for calculating "::x and ::y" (i.e. all the ancestors that
-    are common to both x and y) in an easy and optimized way. We can't quite
-    use "::head()" because that revset returns "::x + ::y + ..." for each head
-    in the repo (whereas we want "::x *and* ::y").
-
     """
     startrevs = getset(repo, fullreposet(repo), x, order=anyorder)
     if not startrevs:



To: valentin.gatienbaron, #hg-reviewers
Cc: mjpieters, mercurial-devel


More information about the Mercurial-devel mailing list