Mercurial > hg
diff tests/test-revset.t @ 38705:e4b270a32ba8
revset: special case commonancestors(none()) to be empty set
This matches the behavior of ancestor(none()).
From an implementation perspective, ancestor() and commonancestors() are
intersection, and ancestors() is union, so it would make some sense that
commonancestors(none()) returned all revisions. However, ancestor(none())
isn't implemented as such, which breaks ancestor(x) == max(commonancestors(x)).
From a user perspective, ancestors of nothing is nothing whichever type
of operation the ancestor predicate does.
author | Yuya Nishihara <yuya@tcha.org> |
---|---|
date | Thu, 12 Jul 2018 23:07:29 +0900 |
parents | 5460926352ee |
children | 7848f284b211 f28812c708a5 |
line wrap: on
line diff
--- a/tests/test-revset.t Tue Jul 10 23:01:53 2018 +0900 +++ b/tests/test-revset.t Thu Jul 12 23:07:29 2018 +0900 @@ -1063,6 +1063,12 @@ 8 9 +test ancestor variants of empty revision + + $ log 'ancestor(none())' + $ log 'ancestors(none())' + $ log 'commonancestors(none())' + test ancestors with depth limit (depth=0 selects the node itself)