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

login
register
mail settings
Submitter phabricator
Date April 29, 2019, 10:52 p.m.
Message ID <1a163cbe0e5e1df7108da792b794177f@localhost.localdomain>
Download mbox | patch
Permalink /patch/39898/
State Not Applicable
Headers show

Comments

phabricator - April 29, 2019, 10:52 p.m.
martinvonz updated this revision to Diff 14970.

REPOSITORY
  rHG Mercurial

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

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