diff mercurial/templatekw.py @ 25700:0fca47b206f6

templatekw: use a list of tags in getlatesttags() instead of joining them

This will be used in the next patch.

It also points out that the documentation for '{latesttag}' is not quite
accurate, since it says "most recent global tag" (singular).  I assume it is too
radical of a change to convert it to a list of strings.  At least ':' is
currently a reserved character in tag names.

--- a/mercurial/templatekw.py	Mon Jun 29 17:19:58 2015 -0700
+++ b/mercurial/templatekw.py	Fri Jun 26 23:23:10 2015 -0400
@@ -120,7 +120,7 @@
     if 'latesttags' not in cache:
         # Cache mapping from rev to a tuple with tag date, tag
         # distance and tag name
-        cache['latesttags'] = {-1: (0, 0, 'null')}
+        cache['latesttags'] = {-1: (0, 0, ['null'])}
     latesttags = cache['latesttags']
 
     rev = ctx.rev()
@@ -133,7 +133,7 @@
         tags = [t for t in ctx.tags()
                 if (repo.tagtype(t) and repo.tagtype(t) != 'local')]
         if tags:
-            latesttags[rev] = ctx.date()[0], 0, ':'.join(sorted(tags))
+            latesttags[rev] = ctx.date()[0], 0, [t for t in sorted(tags)]
             continue
         try:
             # The tuples are laid out so the right one can be found by
@@ -328,7 +328,7 @@
     """:latesttag: String. Most recent global tag in the ancestors of this
     changeset.
     """
-    return getlatesttags(repo, ctx, cache)[2]
+    return ':'.join(getlatesttags(repo, ctx, cache)[2])
 
 def showlatesttagdistance(repo, ctx, templ, cache, **args):
     """:latesttagdistance: Integer. Longest path to the latest tag."""