Patchwork D6275: copies: delete a redundant cleanup step in _chain()

login
register
mail settings
Submitter phabricator
Date April 18, 2019, 5:58 p.m.
Message ID <b427c8e0166f65f302a540204fd6b278@localhost.localdomain>
Download mbox | patch
Permalink /patch/39748/
State Not Applicable
Headers show

Comments

phabricator - April 18, 2019, 5:58 p.m.
martinvonz updated this revision to Diff 14847.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D6275?vs=14844&id=14847

REVISION DETAIL
  https://phab.mercurial-scm.org/D6275

AFFECTED FILES
  mercurial/copies.py

CHANGE DETAILS




To: martinvonz, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/mercurial/copies.py b/mercurial/copies.py
--- a/mercurial/copies.py
+++ b/mercurial/copies.py
@@ -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