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