filelog: stop proxying checksize() (API)
authorGregory Szorc <gregory.szorc@gmail.com>
Wed, 19 Sep 2018 11:22:56 -0700
changeset 39844 0cb3e02e1d1b
parent 39843 68282a7b29a7
child 39845 e6d3d39cc1c7
filelog: stop proxying checksize() (API) This was only used by verify code. And the check using it is now implemented as part of verifyintegrity(). The method is unused and is revlog-centric, which means it isn't appropriate for the file storage interface. So remove it. Differential Revision: https://phab.mercurial-scm.org/D4703
mercurial/filelog.py
mercurial/repository.py
tests/simplestorerepo.py
--- a/mercurial/filelog.py	Wed Sep 19 11:20:02 2018 -0700
+++ b/mercurial/filelog.py	Wed Sep 19 11:22:56 2018 -0700
@@ -121,10 +121,6 @@
     def files(self):
         return self._revlog.files()
 
-    # Used by verify.
-    def checksize(self):
-        return self._revlog.checksize()
-
     def read(self, node):
         t = self.revision(node)
         if not t.startswith('\1\n'):
--- a/mercurial/repository.py	Wed Sep 19 11:20:02 2018 -0700
+++ b/mercurial/repository.py	Wed Sep 19 11:22:56 2018 -0700
@@ -751,12 +751,6 @@
         be a better API for that.
         """
 
-    def checksize():
-        """Obtain the expected sizes of backing files.
-
-        TODO this is used by verify and it should not be part of the interface.
-        """
-
     def verifyintegrity(state):
         """Verifies the integrity of file storage.
 
--- a/tests/simplestorerepo.py	Wed Sep 19 11:20:02 2018 -0700
+++ b/tests/simplestorerepo.py	Wed Sep 19 11:22:56 2018 -0700
@@ -422,10 +422,6 @@
 
         return [b'/'.join((self._storepath, f)) for f in entries]
 
-    # Required by verify.
-    def checksize(self):
-        return 0, 0
-
     def add(self, text, meta, transaction, linkrev, p1, p2):
         if meta or text.startswith(b'\1\n'):
             text = revlog.packmeta(meta, text)