revlog: drop the _generaldelta attribute from the imanifeststorage interface
authorPierre-Yves David <pierre-yves.david@octobus.net>
Tue, 10 Oct 2023 10:58:28 +0200
changeset 51047 861fe9273d57
parent 51046 f636103c4d67
child 51048 59c6f99723b1
revlog: drop the _generaldelta attribute from the imanifeststorage interface It never belonged here.
hgext/remotefilelog/remotefilelog.py
mercurial/bundlerepo.py
mercurial/interfaces/repository.py
mercurial/manifest.py
mercurial/unionrepo.py
--- a/hgext/remotefilelog/remotefilelog.py	Tue Oct 10 10:56:42 2023 +0200
+++ b/hgext/remotefilelog/remotefilelog.py	Tue Oct 10 10:58:28 2023 +0200
@@ -44,7 +44,6 @@
 
 class remotefilelog:
 
-    _generaldelta = True
     _flagserrorclass = error.RevlogError
 
     def __init__(self, opener, path, repo):
--- a/mercurial/bundlerepo.py	Tue Oct 10 10:56:42 2023 +0200
+++ b/mercurial/bundlerepo.py	Tue Oct 10 10:58:28 2023 +0200
@@ -235,15 +235,6 @@
             )
         return super(bundlemanifest, self).dirlog(d)
 
-    # XXX small hack to work around the use of manifest.manifestrevlog
-    @property
-    def _generaldelta(self):
-        return self._revlog.delta_config.general_delta
-
-    @_generaldelta.setter
-    def _generaldelta(self, value):
-        self._revlog.delta_config.general_delta = value
-
 
 class bundlefilelog(filelog.filelog):
     def __init__(self, opener, path, cgunpacker, linkmapper):
--- a/mercurial/interfaces/repository.py	Tue Oct 10 10:56:42 2023 +0200
+++ b/mercurial/interfaces/repository.py	Tue Oct 10 10:58:28 2023 +0200
@@ -1229,13 +1229,6 @@
         """
     )
 
-    _generaldelta = interfaceutil.Attribute(
-        """Whether generaldelta storage is being used.
-
-        TODO this is revlog specific and should not be exposed.
-        """
-    )
-
     fulltextcache = interfaceutil.Attribute(
         """Dict with cache of fulltexts.
 
--- a/mercurial/manifest.py	Tue Oct 10 10:56:42 2023 +0200
+++ b/mercurial/manifest.py	Tue Oct 10 10:58:28 2023 +0200
@@ -1614,7 +1614,6 @@
         )
 
         self.index = self._revlog.index
-        self._generaldelta = self._revlog.delta_config.general_delta
 
     def get_revlog(self):
         """return an actual revlog instance if any
--- a/mercurial/unionrepo.py	Tue Oct 10 10:56:42 2023 +0200
+++ b/mercurial/unionrepo.py	Tue Oct 10 10:58:28 2023 +0200
@@ -213,15 +213,6 @@
             self, opener, self._revlog.radix, manifest2, linkmapper
         )
 
-    # XXX small hack to work around the use of manifest.manifestrevlog
-    @property
-    def _generaldelta(self):
-        return self._revlog.delta_config.general_delta
-
-    @_generaldelta.setter
-    def _generaldelta(self, value):
-        self._revlog.delta_config.general_delta = value
-
 
 class unionfilelog(filelog.filelog):
     def __init__(self, opener, path, opener2, linkmapper, repo):