Patchwork D3156: githelp: use revsymbol() for looking up symbol

login
register
mail settings
Submitter phabricator
Date April 6, 2018, 12:41 p.m.
Message ID <2b0107f697024d6c490d8bca839fd206@localhost.localdomain>
Download mbox | patch
Permalink /patch/30432/
State Not Applicable
Headers show

Comments

phabricator - April 6, 2018, 12:41 p.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG764ada920db5: githelp: use revsymbol() for looking up symbol (authored by martinvonz, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D3156?vs=7771&id=7778

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

AFFECTED FILES
  hgext/githelp.py

CHANGE DETAILS




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

Patch

diff --git a/hgext/githelp.py b/hgext/githelp.py
--- a/hgext/githelp.py
+++ b/hgext/githelp.py
@@ -26,6 +26,7 @@ 
     error,
     fancyopts,
     registrar,
+    scmutil,
 )
 from mercurial.utils import (
     procutil,
@@ -253,7 +254,7 @@ 
     too many ways to spell revisions in git for us to reasonably catch all of
     them, so let's be conservative.
     """
-    if string in repo:
+    if scmutil.isrevsymbol(repo, string):
         # if it's definitely a revision let's not even check if a file of the
         # same name exists.
         return False