copies: return True instead of filename as it is expected to return boolean
authorSushil khanchi <sushilkhanchi97@gmail.com>
Thu, 14 Feb 2019 18:28:27 +0530
changeset 41724 35158796f52f
parent 41723 ebbc4e70ebd1
child 41727 e1643a0455c8
copies: return True instead of filename as it is expected to return boolean As the function documentation says this function should True if we hit a match and False otherwise. And I see that we are not using that returned filename anywhere. Differential Revision: https://phab.mercurial-scm.org/D5964
mercurial/copies.py
--- a/mercurial/copies.py	Mon Feb 11 19:41:37 2019 +0300
+++ b/mercurial/copies.py	Thu Feb 14 18:28:27 2019 +0530
@@ -776,7 +776,7 @@
     """
 
     if f1 == f2:
-        return f1 # a match
+        return True # a match
 
     g1, g2 = f1.ancestors(), f2.ancestors()
     try: