findcopies: fix rename bug
authorMatt Mackall <mpm@selenic.com>
Mon, 08 Oct 2007 18:47:22 -0500
changeset 5429 77cafe8f89e0
parent 5428 eb1b6aaeb32e
child 5430 0bdea0abe62e
findcopies: fix rename bug We've fiddled with this line several times, and an old bug has reappeared from it. Let's take a peek at the history. The original "or" (rev 3674, in 0.9.2 and 0.9.3): http://www.selenic.com/hg/rev/9103dab96093 Then I changed it to an "and" to fix a bug (rev 4304): http://www.selenic.com/hg/rev/4787e2b0dd03 Then for reasons now lost in the mists of time, I dropped half (rev 4399): http://www.selenic.com/hg/rev/93652499bed3 Then we added back the "or" (rev 4416, in 0.9.4): http://www.selenic.com/hg/rev/bb1800a7d7e1 So it seems it ought to be "and".
mercurial/merge.py
--- a/mercurial/merge.py	Mon Oct 08 18:47:21 2007 -0500
+++ b/mercurial/merge.py	Mon Oct 08 18:47:22 2007 -0500
@@ -180,7 +180,7 @@
                 continue
             # named changed on only one side?
             if ca.path() == c.path() or ca.path() == c2.path():
-                if c == ca or c2 == ca: # no merge needed, ignore copy
+                if c == ca and c2 == ca: # no merge needed, ignore copy
                     continue
                 copy[c.path()] = of
 
@@ -280,7 +280,7 @@
 
 def symmetricdifference(repo, rev1, rev2):
     """symmetric difference of the sets of ancestors of rev1 and rev2
-    
+
     I.e. revisions that are ancestors of rev1 or rev2, but not both.
     """
     # basic idea: