Page MenuHomePhabricator

changelog: fix handling of empty copy entries in changeset
ClosedPublic

Authored by martinvonz on Mon, Jul 1, 7:28 PM.

Details

Summary

Before this patch, when an empty value was found in the changeset, we
would get a ValueError, which would result in None being returned for
addedfiles/removedfiles and p1copies/p2copies. That made 278dcb24e535
(copies: write empty entries in changeset when also writing to
filelog, 2019-04-23) ineffective at helping the read path not look for
copies in the filelogs.

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

martinvonz created this revision.Mon, Jul 1, 7:28 PM
pulkit accepted this revision.Tue, Jul 2, 6:28 PM
This revision is now accepted and ready to land.Tue, Jul 2, 6:28 PM