diff mercurial/templatefuncs.py @ 37708:8e8541610d85

scmutil: make shortesthexnodeidprefix() use unfiltered repo Both callers were doing this, and resolvehexnodeidprefix() was also working on the unfiltered repo, so it makes more sense to have it all in one place. Differential Revision: https://phab.mercurial-scm.org/D3313
author Martin von Zweigbergk <martinvonz@google.com>
date Fri, 13 Apr 2018 10:36:03 -0700
parents e743b8524d60
children 7b2955624777
line wrap: on
line diff
--- a/mercurial/templatefuncs.py	Sun Apr 15 19:41:34 2018 +0800
+++ b/mercurial/templatefuncs.py	Fri Apr 13 10:36:03 2018 -0700
@@ -587,11 +587,8 @@
                                 # i18n: "shortest" is a keyword
                                 _("shortest() expects an integer minlength"))
 
-    # _partialmatch() of filtered changelog could take O(len(repo)) time,
-    # which would be unacceptably slow. so we look for hash collision in
-    # unfiltered space, which means some hashes may be slightly longer.
     repo = context.resource(mapping, 'ctx')._repo
-    return scmutil.shortesthexnodeidprefix(repo.unfiltered(), node, minlength)
+    return scmutil.shortesthexnodeidprefix(repo, node, minlength)
 
 @templatefunc('strip(text[, chars])')
 def strip(context, mapping, args):