# HG changeset patch # User Gregory Szorc # Date 1537381376 25200 # Node ID 0cb3e02e1d1bf21b87c8a8b7d4c3cc2065b211a7 # Parent 68282a7b29a7dc18d8e944619d2cf503bd5d9400 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 diff -r 68282a7b29a7 -r 0cb3e02e1d1b mercurial/filelog.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'): diff -r 68282a7b29a7 -r 0cb3e02e1d1b mercurial/repository.py --- 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. diff -r 68282a7b29a7 -r 0cb3e02e1d1b tests/simplestorerepo.py --- 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)