Page MenuHomePhabricator

copies: delete a redundant cleanup step in _chain()
ClosedPublic

Authored by martinvonz on Apr 18 2019, 12:55 PM.

Details

Summary

The check is redundant since d5edb5d3a337 (copies: filter out copies
when target is not in destination manifest, 2019-02-14). To test that
hypothesis, I made this change in the commit that commit, but all
tests still passed. I think the case was necessary before then, we
just didn't have tests for it.

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.