This is an archive of the discontinued Mercurial Phabricator instance.

revlogutils: move the NodeMap class in a dedicated nodemap module
ClosedPublic

Authored by marmoute on Jan 7 2020, 7:52 AM.

Details

Summary

We will introduce more nodemap related code, so it make sense to move the
existing code in that module first.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

marmoute created this revision.Jan 7 2020, 7:52 AM
This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.

We recently gained a mercurial.utils module. Should mercurial.revlogutils become mercurial.utils.revlogutil instead?

utils is older than revlogutils, revlogutils focus on revlog related changes and his worth keeping at this.