( )⚙ D1779 unionrepo: use named attributes on revlog index entries

This is an archive of the discontinued Mercurial Phabricator instance.

unionrepo: use named attributes on revlog index entries
AcceptedPublic

Authored by indygreg on Dec 26 2017, 7:36 PM.

Details

Reviewers
durin42
Group Reviewers
hg-reviewers
Summary

This appears to be the last user of the offsets in core (as far as
the test harness was able to reveal).

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

indygreg created this revision.Dec 26 2017, 7:36 PM
durin42 accepted this revision.Jan 10 2018, 5:54 PM
durin42 added a subscriber: durin42.

Happy to see these land, but waiting for any objections from others.

This revision is now accepted and ready to land.Jan 10 2018, 5:54 PM

I think we should hold off landing until the native C type is ready. I don't like shipping a performance regression on such low-level code without any other user benefit to offset it. The assumption here is that I'm the only one who will absorb pain from this series lingering. I'm willing to take that hit.