Patchwork D6560: copies: remove a redundant matcher filtering in _changesetforwardcopies()

login
register
mail settings
Submitter phabricator
Date June 24, 2019, 6:59 p.m.
Message ID <e0de72668c2c9520baf14ea8288b5fb8@localhost.localdomain>
Download mbox | patch
Permalink /patch/40667/
State Not Applicable
Headers show

Comments

phabricator - June 24, 2019, 6:59 p.m.
Closed by commit rHGc7104896ec42: copies: remove a redundant matcher filtering in _changesetforwardcopies() (authored by martinvonz).
This revision was automatically updated to reflect the committed changes.
This revision was not accepted when it landed; it landed in state "Needs Review".

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D6560?vs=15624&id=15650

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D6560/new/

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

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
@@ -279,8 +279,6 @@ 
             copies = {}
             allcopies = set(copies1) | set(copies2)
             for dst in allcopies:
-                if not alwaysmatch and not match(dst):
-                    continue
                 # Unlike when copies are stored in the filelog, we consider
                 # it a copy even if the destination already existed on the
                 # other branch. It's simply too expensive to check if the