copies: delete a redundant cleanup step in _chain()
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.
Differential Revision: https://phab.mercurial-scm.org/D6275
--- a/mercurial/copies.py Wed Apr 17 23:10:14 2019 -0700
+++ b/mercurial/copies.py Thu Apr 18 00:05:05 2019 -0700
@@ -134,10 +134,6 @@
if t[v] != k:
# file wasn't renamed back to itself (i.e. case 4, not 3)
t[k] = t[v]
- if v not in dst:
- # chain was a rename, not a copy
- # this deletes the copy for 'y' in case 4
- del t[v]
if v in src:
# file is a copy of an existing file, i.e. case 6.
t[k] = v