Patchwork D6810: revlog: use the new sidedata map return in the sidedata method

login
register
mail settings
Submitter phabricator
Date Sept. 27, 2019, 3:05 p.m.
Message ID <7f6e06f0e5d75f94402bf31ceccc92ce@localhost.localdomain>
Download mbox | patch
Permalink /patch/41782/
State Not Applicable
Headers show

Comments

phabricator - Sept. 27, 2019, 3:05 p.m.
Closed by commit rHG9d62f9fa332f: revlog: use the new sidedata map return in the sidedata method (authored by marmoute).
This revision was automatically updated to reflect the committed changes.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D6810?vs=16491&id=16630

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D6810/new/

REVISION DETAIL
  https://phab.mercurial-scm.org/D6810

AFFECTED FILES
  mercurial/revlog.py

CHANGE DETAILS




To: marmoute, yuja, durin42, indygreg, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/mercurial/revlog.py b/mercurial/revlog.py
--- a/mercurial/revlog.py
+++ b/mercurial/revlog.py
@@ -1623,8 +1623,7 @@ 
         mapping object will likely be used in the future for a more
         efficient/lazy code.
         """
-        # XXX will actualy return data once storage is implemented.
-        return {}
+        return self._revisiondata(nodeorrev, _df)[1]
 
     def _revisiondata(self, nodeorrev, _df=None, raw=False):
         # deal with <nodeorrev> argument type