Page MenuHomePhabricator

branchmap: micro-optimize branchinfo

Authored by joerg.sonnenberger on Dec 17 2020, 7:30 AM.



changelogrevision() is supposed to be used if not all data of is used. This is the case here as only the extra field is
used. This also improves extensibility as at least hgext.git doesn't

Diff Detail

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

Event Timeline

You description mention micro-optimisation, but I don't see any number. What's the performance impact of that ? nowadays is literally changelog.changelogrevision followed by creating a tuple from its parts. So this step just skips creating the tuple with unnecessary elements.

marmoute accepted this revision.Fri, Jan 8, 11:04 AM
pulkit accepted this revision.Sat, Jan 9, 5:04 AM
This revision is now accepted and ready to land.Sat, Jan 9, 5:04 AM
This revision was automatically updated to reflect the committed changes.