Mercurial > hg
changeset 39844:0cb3e02e1d1b
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
| author | Gregory Szorc <gregory.szorc@gmail.com> |
|---|---|
| date | Wed, 19 Sep 2018 11:22:56 -0700 |
| parents | 68282a7b29a7 |
| children | e6d3d39cc1c7 |
| files | mercurial/filelog.py mercurial/repository.py tests/simplestorerepo.py |
| diffstat | 3 files changed, 0 insertions(+), 14 deletions(-) [+] |
line wrap: on
line diff
--- 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)