copies: move `merged` testing sooner
authorPierre-Yves David <pierre-yves.david@octobus.net>
Thu, 01 Oct 2020 09:45:00 +0200
changeset 45673 7990e7d957b0
parent 45672 f877b3628015
child 45674 66cb7ad3787c
copies: move `merged` testing sooner previously `is_merged` was an expensive callback. Now that all this data is pre-computed, this is is a simple membership testing. So it is probably cheaper than the membership testing. Differential Revision: https://phab.mercurial-scm.org/D9141
mercurial/copies.py
--- a/mercurial/copies.py	Thu Oct 01 09:42:39 2020 +0200
+++ b/mercurial/copies.py	Thu Oct 01 09:45:00 2020 +0200
@@ -397,9 +397,9 @@
                 and dest in changes.salvaged
             ):
                 minor[dest] = value
-            elif not isancestor(new_tt, other_tt):
+            elif changes is not None and dest in changes.merged:
                 minor[dest] = value
-            elif changes is not None and dest in changes.merged:
+            elif not isancestor(new_tt, other_tt):
                 minor[dest] = value