Mercurial > hg-stable
changeset 42258:df7ad90edbd4
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
author | Martin von Zweigbergk <martinvonz@google.com> |
---|---|
date | Thu, 18 Apr 2019 00:05:05 -0700 |
parents | d1c2688eda80 |
children | 5a3979529740 |
files | mercurial/copies.py |
diffstat | 1 files changed, 0 insertions(+), 4 deletions(-) [+] |
line wrap: on
line diff
--- 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