This is an archive of the discontinued Mercurial Phabricator instance.

revlog: inline opener options logic into _loadindex()
ClosedPublic

Authored by indygreg on Jan 10 2019, 7:32 PM.

Details

Summary

We always call _loadindex() during init. But we also call
_loadindex() as part of censorrevision(). Before, when reloading
the index during censorrevision(), we would lose the configured
mmapindexthreshold setting from the opener. By inlining the
logic in _loadindex(), we ensure that opener options are always
respected when loading the index.

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

indygreg created this revision.Jan 10 2019, 7:32 PM
This revision was automatically updated to reflect the committed changes.