fileset: remove callexisting flag and mctx.existing() (API)
authorYuya Nishihara <yuya@tcha.org>
Sat, 09 Jun 2018 22:31:51 +0900
changeset 38690 5d9749c598f0
parent 38689 ff5b6fca1082
child 38691 370ff3e34160
fileset: remove callexisting flag and mctx.existing() (API) They are no longer needed since any files are included as long as they are passed in to the matcher.
mercurial/fileset.py
mercurial/registrar.py
tests/test-fileset.t
--- a/mercurial/fileset.py	Sat Jun 09 19:55:10 2018 +0900
+++ b/mercurial/fileset.py	Sat Jun 09 22:31:51 2018 +0900
@@ -181,12 +181,7 @@
 def func(mctx, a, b):
     funcname = getsymbol(a)
     if funcname in symbols:
-        enabled = mctx._existingenabled
-        mctx._existingenabled = funcname in _existingcallers
-        try:
-            return symbols[funcname](mctx, b)
-        finally:
-            mctx._existingenabled = enabled
+        return symbols[funcname](mctx, b)
 
     keep = lambda fn: getattr(fn, '__doc__', None) is not None
 
@@ -203,9 +198,6 @@
 # filesets using matchctx.status()
 _statuscallers = set()
 
-# filesets using matchctx.existing()
-_existingcallers = set()
-
 predicate = registrar.filesetpredicate()
 
 @predicate('modified()', callstatus=True)
@@ -285,7 +277,7 @@
     getargs(x, 0, 0, _("tracked takes no arguments"))
     return mctx.predicate(mctx.ctx.__contains__, predrepr='tracked')
 
-@predicate('binary()', callexisting=True)
+@predicate('binary()')
 def binary(mctx, x):
     """File that appears to be binary (contains NUL bytes).
     """
@@ -294,7 +286,7 @@
     return mctx.fpredicate(lambda fctx: fctx.isbinary(),
                            predrepr='binary', cache=True)
 
-@predicate('exec()', callexisting=True)
+@predicate('exec()')
 def exec_(mctx, x):
     """File that is marked as executable.
     """
@@ -303,7 +295,7 @@
     ctx = mctx.ctx
     return mctx.predicate(lambda f: ctx.flags(f) == 'x', predrepr='exec')
 
-@predicate('symlink()', callexisting=True)
+@predicate('symlink()')
 def symlink(mctx, x):
     """File that is marked as a symlink.
     """
@@ -354,7 +346,7 @@
     return mctx.predicate(lambda f: util.checkwinfilename(f) is None,
                           predrepr='portable')
 
-@predicate('grep(regex)', callexisting=True)
+@predicate('grep(regex)')
 def grep(mctx, x):
     """File contains the given regular expression.
     """
@@ -408,7 +400,7 @@
         b = _sizetomax(expr)
         return lambda x: x >= a and x <= b
 
-@predicate('size(expression)', callexisting=True)
+@predicate('size(expression)')
 def size(mctx, x):
     """File size matches the given expression. Examples:
 
@@ -423,7 +415,7 @@
     return mctx.fpredicate(lambda fctx: m(fctx.size()),
                            predrepr=('size(%r)', expr), cache=True)
 
-@predicate('encoding(name)', callexisting=True)
+@predicate('encoding(name)')
 def encoding(mctx, x):
     """File can be successfully decoded with the given character
     encoding. May not be useful for encodings other than ASCII and
@@ -445,7 +437,7 @@
 
     return mctx.fpredicate(encp, predrepr=('encoding(%r)', enc), cache=True)
 
-@predicate('eol(style)', callexisting=True)
+@predicate('eol(style)')
 def eol(mctx, x):
     """File contains newlines of the given style (dos, unix, mac). Binary
     files are excluded, files with mixed line endings match multiple
@@ -566,7 +558,7 @@
         self.subset = subset
         self._status = status
         self._badfn = badfn
-        self._existingenabled = False
+
     def status(self):
         return self._status
 
@@ -621,17 +613,6 @@
 
     def filter(self, files):
         return [f for f in files if f in self.subset]
-    def existing(self):
-        if not self._existingenabled:
-            raise error.ProgrammingError('unexpected existing() invocation')
-        if self._status is not None:
-            removed = set(self._status[3])
-            unknown = set(self._status[4] + self._status[5])
-        else:
-            removed = set()
-            unknown = set()
-        return (f for f in self.subset
-                if (f in self.ctx and f not in removed) or f in unknown)
 
     def switch(self, ctx, status=None):
         subset = self.filter(_buildsubset(ctx, status))
@@ -683,13 +664,7 @@
 def _buildstatus(ctx, tree, basectx=None):
     # do we need status info?
 
-    # temporaty boolean to simplify the next conditional
-    purewdir = ctx.rev() is None and basectx is None
-
-    if (_intree(_statuscallers, tree) or
-        # Using matchctx.existing() on a workingctx requires us to check
-        # for deleted files.
-        (purewdir and _intree(_existingcallers, tree))):
+    if _intree(_statuscallers, tree):
         unknown = _intree(['unknown'], tree)
         ignored = _intree(['ignored'], tree)
 
@@ -711,10 +686,8 @@
         symbols[name] = func
         if func._callstatus:
             _statuscallers.add(name)
-        if func._callexisting:
-            _existingcallers.add(name)
 
-# load built-in predicates explicitly to setup _statuscallers/_existingcallers
+# load built-in predicates explicitly to setup _statuscallers
 loadpredicate(None, None, predicate)
 
 # tell hggettext to extract docstrings from these functions:
--- a/mercurial/registrar.py	Sat Jun 09 19:55:10 2018 +0900
+++ b/mercurial/registrar.py	Sat Jun 09 22:31:51 2018 +0900
@@ -247,10 +247,6 @@
      implies 'matchctx.status()' at runtime or not (False, by
      default).
 
-    Optional argument 'callexisting' indicates whether a predicate
-    implies 'matchctx.existing()' at runtime or not (False, by
-    default).
-
     'filesetpredicate' instance in example above can be used to
     decorate multiple functions.
 
@@ -263,9 +259,8 @@
     _getname = _funcregistrarbase._parsefuncdecl
     _docformat = "``%s``\n    %s"
 
-    def _extrasetup(self, name, func, callstatus=False, callexisting=False):
+    def _extrasetup(self, name, func, callstatus=False):
         func._callstatus = callstatus
-        func._callexisting = callexisting
 
 class _templateregistrarbase(_funcregistrarbase):
     """Base of decorator to register functions as template specific one
--- a/tests/test-fileset.t	Sat Jun 09 19:55:10 2018 +0900
+++ b/tests/test-fileset.t	Sat Jun 09 22:31:51 2018 +0900
@@ -448,27 +448,6 @@
   mixed
   unknown
 
-Test detection of unintentional 'matchctx.existing()' invocation
-
-  $ cat > $TESTTMP/existingcaller.py <<EOF
-  > from mercurial import registrar
-  > 
-  > filesetpredicate = registrar.filesetpredicate()
-  > @filesetpredicate(b'existingcaller()', callexisting=False)
-  > def existingcaller(mctx, x):
-  >     # this 'mctx.existing()' invocation is unintentional
-  >     existing = set(mctx.existing())
-  >     return mctx.predicate(existing.__contains__, cache=False)
-  > EOF
-
-  $ cat >> .hg/hgrc <<EOF
-  > [extensions]
-  > existingcaller = $TESTTMP/existingcaller.py
-  > EOF
-
-  $ fileset 'existingcaller()' 2>&1 | tail -1
-  *ProgrammingError: *unexpected existing() invocation* (glob)
-
 Test 'revs(...)'
 ================