Patchwork D5964: copies: return True instead of filename as it is expected to return boolean

login
register
mail settings
Submitter phabricator
Date Feb. 19, 2019, 6:13 a.m.
Message ID <c8ba4fffee57b4db9b0d7f3f6389fbc6@localhost.localdomain>
Download mbox | patch
Permalink /patch/38819/
State Not Applicable
Headers show

Comments

phabricator - Feb. 19, 2019, 6:13 a.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG35158796f52f: copies: return True instead of filename as it is expected to return boolean (authored by khanchi97, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D5964?vs=14093&id=14142

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

AFFECTED FILES
  mercurial/copies.py

CHANGE DETAILS




To: khanchi97, #hg-reviewers, martinvonz
Cc: mercurial-devel

Patch

diff --git a/mercurial/copies.py b/mercurial/copies.py
--- a/mercurial/copies.py
+++ b/mercurial/copies.py
@@ -776,7 +776,7 @@ 
     """
 
     if f1 == f2:
-        return f1 # a match
+        return True # a match
 
     g1, g2 = f1.ancestors(), f2.ancestors()
     try: