Page MenuHomePhabricator

dirstatemap: fix copymap.pop in Rust to return the value it pops
ClosedPublic

Authored by marmoute on Oct 2 2021, 11:05 AM.

Details

Summary

I guess this was overlooked in the initial implementation?

Without this, the next patch would, loose copy information in setparent.

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.Oct 2 2021, 11:05 AM
Alphare added a subscriber: Alphare.Oct 3 2021, 4:35 PM

Oops! Weird that this didn't show up in a test.

Alphare accepted this revision.Oct 3 2021, 4:35 PM
This revision is now accepted and ready to land.Oct 3 2021, 4:35 PM