Page MenuHomePhabricator

revlog: rename `versionflags` to header
ClosedPublic

Authored by marmoute on Mon, May 3, 7:50 AM.

Details

Summary

The variable is named as such because it contains "version" information and
"flags" information. However you mostly needs to know the code to understand the
name which is not great. The fact that this is the very first four bytes in all
revlog seems more relevant, so we rename the variable "header".

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.Mon, May 3, 7:50 AM
baymax updated this revision to Diff 27511.Mon, May 3, 9:44 PM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

Alphare accepted this revision.Tue, May 11, 10:09 AM
This revision is now accepted and ready to land.Tue, May 11, 10:09 AM
This revision was automatically updated to reflect the committed changes.