D3312: revlog: move shortest() to scmutil.shortesthexnodeidprefix() (API)

martinvonz (Martin von Zweigbergk) phabricator at mercurial-scm.org
Sat Apr 14 00:09:40 EDT 2018


martinvonz added a comment.


  In https://phab.mercurial-scm.org/D3312#53423, @yuja wrote:
  
  > IIRC, @quark said it's in revlog because the implementation may vary
  >  depending on storage, such as looking up in radix tree and rewind?
  
  
  That's reasonable. Perhaps I'll just make shortesthexnodeidprefix() a thin wrapper then, so all it does (at the end of this series) is to delegate to the unfiltered repo. Does that seem okay with you?

REPOSITORY
  rHG Mercurial

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

To: martinvonz, indygreg, #hg-reviewers
Cc: yuja, quark, mercurial-devel


More information about the Mercurial-devel mailing list