This is an archive of the discontinued Mercurial Phabricator instance.

revlog: reset _nodepos after strip
ClosedPublic

Authored by joerg.sonnenberger on Apr 7 2018, 7:09 PM.

Details

Summary

When using the pure revlog parser, _nodepos is used to keep track of the
position during index scanning in the non-cached cache. If it is out of
bounds, BaseIndexObject._fix_index will assert. Since strip can actually
remove the position scanned last, make sure to reset it. Add an
assertion in the place where the invariance is clearer.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

durin42 accepted this revision.Apr 9 2018, 4:09 PM
This revision is now accepted and ready to land.Apr 9 2018, 4:09 PM
This revision was automatically updated to reflect the committed changes.