# HG changeset patch # User Martin von Zweigbergk # Date 1414597419 25200 # Node ID 55c449345b103017ea464d474fa9a2650529eac7 # Parent d44d53bc9a1ea53709c7d8685a08e323ab4724e0 match: add isexact() method to hide internals Comparing a function reference seems bad. diff -r d44d53bc9a1e -r 55c449345b10 mercurial/dirstate.py --- a/mercurial/dirstate.py Wed Nov 19 15:56:58 2014 -0800 +++ b/mercurial/dirstate.py Wed Oct 29 08:43:39 2014 -0700 @@ -611,7 +611,7 @@ dirsnotfound = [] notfoundadd = dirsnotfound.append - if match.matchfn != match.exact and self._checkcase: + if not match.isexact() and self._checkcase: normalize = self._normalize else: normalize = None @@ -711,7 +711,7 @@ join = self._join exact = skipstep3 = False - if matchfn == match.exact: # match.exact + if match.isexact(): # match.exact exact = True dirignore = util.always # skip step 2 elif match.files() and not match.anypats(): # match.match, no patterns @@ -912,7 +912,7 @@ if match.always(): return dmap.keys() files = match.files() - if match.matchfn == match.exact: + if match.isexact(): # fast path -- filter the other way around, since typically files is # much smaller than dmap return [f for f in files if f in dmap] diff -r d44d53bc9a1e -r 55c449345b10 mercurial/manifest.py --- a/mercurial/manifest.py Wed Nov 19 15:56:58 2014 -0800 +++ b/mercurial/manifest.py Wed Oct 29 08:43:39 2014 -0700 @@ -164,7 +164,7 @@ return self.copy() files = match.files() - if (len(files) < 100 and (match.matchfn == match.exact or + if (len(files) < 100 and (match.isexact() or (not match.anypats() and util.all(fn in self for fn in files)))): return self.intersectfiles(files) @@ -519,7 +519,7 @@ return self.copy() files = match.files() - if (match.matchfn == match.exact or + if (match.isexact() or (not match.anypats() and util.all(fn in self for fn in files))): return self.intersectfiles(files) diff -r d44d53bc9a1e -r 55c449345b10 mercurial/match.py --- a/mercurial/match.py Wed Nov 19 15:56:58 2014 -0800 +++ b/mercurial/match.py Wed Oct 29 08:43:39 2014 -0700 @@ -171,6 +171,9 @@ - optimization might be possible and necessary.''' return self._always + def isexact(self): + return self.matchfn == self.exact + def exact(root, cwd, files): return match(root, cwd, files, exact=True)