# HG changeset patch
# User Kyle Lippincott <spectral@google.com>
# Date 2018-12-27 23:19:46
# Node ID 517a51d9cd7fa88c12eb952391192ef6b079c855
# Parent  8ecb17b7f43234c1eae0fb769dab795bf79fa485

remotefilelog: fix bug in maybesparsematch returning alwaysmatcher

The description of the method says that it should return None if sparse is not
used in this repository; since sparse.matcher() returns alwaysmatcher if sparse
is not enabled, I'm using that as the signal to return None here to preserve the
previous behavior.

Differential Revision: https://phab.mercurial-scm.org/D5487

diff --git a/hgext/remotefilelog/shallowrepo.py b/hgext/remotefilelog/shallowrepo.py
--- a/hgext/remotefilelog/shallowrepo.py
+++ b/hgext/remotefilelog/shallowrepo.py
@@ -143,8 +143,13 @@ def wraprepo(repo):
             sparse repository.
             '''
             if revs:
-                return sparse.matcher(repo, revs=revs)
-            return sparse.matcher(repo)
+                ret = sparse.matcher(repo, revs=revs)
+            else:
+                ret = sparse.matcher(repo)
+
+            if ret.always():
+                return None
+            return ret
 
         def file(self, f):
             if f[0] == '/':