This is an archive of the discontinued Mercurial Phabricator instance.

dirstate: move the copymap drop inside dropfile
ClosedPublic

Authored by marmoute on Sep 14 2021, 5:20 PM.

Details

Summary

Since the copymap is part of the dirstatemap it make more sense for the
dirstatemap to manage it directly.

This is part of a generic effort to move unified logic at lower level and to
clean up higher level API.

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.Sep 14 2021, 5:20 PM
Alphare accepted this revision.Sep 15 2021, 3:59 AM
This revision is now accepted and ready to land.Sep 15 2021, 3:59 AM

Should the same thing be done for _add?

Should the same thing be done for _add?

Yes, I am actually working on _add as we "speak"

This revision was automatically updated to reflect the committed changes.