comparison mercurial/copies.py @ 41724:35158796f52f

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
author Sushil khanchi <sushilkhanchi97@gmail.com>
date Thu, 14 Feb 2019 18:28:27 +0530
parents 75e753a26806
children 012f695546aa
comparison
equal deleted inserted replaced
41723:ebbc4e70ebd1 41724:35158796f52f
774 up the integer comparison logic, hence the pre-step check for 774 up the integer comparison logic, hence the pre-step check for
775 None (f1 and f2 can only be workingfilectx's initially). 775 None (f1 and f2 can only be workingfilectx's initially).
776 """ 776 """
777 777
778 if f1 == f2: 778 if f1 == f2:
779 return f1 # a match 779 return True # a match
780 780
781 g1, g2 = f1.ancestors(), f2.ancestors() 781 g1, g2 = f1.ancestors(), f2.ancestors()
782 try: 782 try:
783 f1r, f2r = f1.linkrev(), f2.linkrev() 783 f1r, f2r = f1.linkrev(), f2.linkrev()
784 784