# HG changeset patch
# User Kyle Lippincott <spectral@google.com>
# Date 1545952786 28800
# 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 -r 8ecb17b7f432 -r 517a51d9cd7f hgext/remotefilelog/shallowrepo.py
--- a/hgext/remotefilelog/shallowrepo.py	Fri Dec 28 12:51:47 2018 -0800
+++ b/hgext/remotefilelog/shallowrepo.py	Thu Dec 27 15:19:46 2018 -0800
@@ -143,8 +143,13 @@
             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] == '/':