bundlerepo: drop the `baserevision` hack
authorPierre-Yves David <pierre-yves.david@octobus.net>
Fri, 30 Aug 2019 17:46:47 +0200
changeset 42822 0f5652f17325
parent 42821 c070ca6ed86d
child 42823 5ba8c328a895
bundlerepo: drop the `baserevision` hack It is not used anywhere anymore, so we can safely drop it.
mercurial/bundlerepo.py
--- a/mercurial/bundlerepo.py	Fri Aug 30 15:04:54 2019 +0200
+++ b/mercurial/bundlerepo.py	Fri Aug 30 17:46:47 2019 +0200
@@ -142,12 +142,6 @@
     def rawdata(self, nodeorrev, _df=None):
         return self.revision(nodeorrev, _df=_df, raw=True)
 
-    def baserevision(self, nodeorrev):
-        # Revlog subclasses may override 'revision' method to modify format of
-        # content retrieved from revlog. To use bundlerevlog with such class one
-        # needs to override 'baserevision' and make more specific call here.
-        return revlog.revlog.rawdata(self, nodeorrev)
-
     def addrevision(self, *args, **kwargs):
         raise NotImplementedError
 
@@ -167,20 +161,6 @@
         bundlerevlog.__init__(self, opener, self.indexfile, cgunpacker,
                               linkmapper)
 
-    def baserevision(self, nodeorrev):
-        # Although changelog doesn't override 'revision' method, some extensions
-        # may replace this class with another that does. Same story with
-        # manifest and filelog classes.
-
-        # This bypasses filtering on changelog.node() and rev() because we need
-        # revision text of the bundle base even if it is hidden.
-        oldfilter = self.filteredrevs
-        try:
-            self.filteredrevs = ()
-            return changelog.changelog.rawdata(self, nodeorrev)
-        finally:
-            self.filteredrevs = oldfilter
-
 class bundlemanifest(bundlerevlog, manifest.manifestrevlog):
     def __init__(self, opener, cgunpacker, linkmapper, dirlogstarts=None,
                  dir=''):
@@ -194,17 +174,6 @@
         self._dirlogstarts = dirlogstarts
         self._linkmapper = linkmapper
 
-    def baserevision(self, nodeorrev):
-        node = nodeorrev
-        if isinstance(node, int):
-            node = self.node(node)
-
-        if node in self.fulltextcache:
-            result = '%s' % self.fulltextcache[node]
-        else:
-            result = manifest.manifestrevlog.rawdata(self, nodeorrev)
-        return result
-
     def dirlog(self, d):
         if d in self._dirlogstarts:
             self.bundle.seek(self._dirlogstarts[d])
@@ -219,9 +188,6 @@
         self._revlog = bundlerevlog(opener, self.indexfile,
                                     cgunpacker, linkmapper)
 
-    def baserevision(self, nodeorrev):
-        return filelog.filelog.rawdata(self, nodeorrev)
-
 class bundlepeer(localrepo.localpeer):
     def canpush(self):
         return False