stack: use repo.revs() instead of revsetlang.formatspec() + scmutil.revrange()
Using scmutil.revrange() it's possible to use multiple revsets at the same
time, but we're not using that functionality in stack.
I thought maybe that function could be used to make stack definition
customizable (by combining various parts into one set), but scmutil.revrange()
gives the union of all provided revsets, which is not very useful in stack's
case (we want "and" between parts, not "or").
--- a/mercurial/stack.py Mon Sep 23 21:29:53 2019 +0900
+++ b/mercurial/stack.py Sun Sep 22 14:33:56 2019 +0700
@@ -7,11 +7,6 @@
from __future__ import absolute_import
-from . import (
- revsetlang,
- scmutil,
-)
-
def getstack(repo, rev=None):
"""return a sorted smartrev of the stack containing either rev if it is
not None or the current working directory parent.
@@ -23,7 +18,6 @@
rev = '.'
revspec = 'only(%s) and not public() and not ::merge()'
- revset = revsetlang.formatspec(revspec, rev)
- revisions = scmutil.revrange(repo, [revset])
+ revisions = repo.revs(revspec, rev)
revisions.sort()
return revisions