templatekw: pass templater to _showlist() by an explicit argument
authorYuya Nishihara <yuya@tcha.org>
Sun, 25 Feb 2018 15:43:35 +0900
changeset 36548 94c4ae452293
parent 36547 7937850a523d
child 36549 a7fbe11a5d59
templatekw: pass templater to _showlist() by an explicit argument Prepares for switching to the (context, mapping) API.
hgext/lfs/__init__.py
mercurial/templatekw.py
--- a/hgext/lfs/__init__.py	Fri Dec 22 21:59:38 2017 +0900
+++ b/hgext/lfs/__init__.py	Sun Feb 25 15:43:35 2018 +0900
@@ -378,7 +378,7 @@
     }
 
     # TODO: make the separator ', '?
-    f = templatekw._showlist('lfs_file', files, args)
+    f = templatekw._showlist('lfs_file', files, args['templ'], args)
     return templatekw._hybrid(f, files, makemap, pycompat.identity)
 
 @command('debuglfsupload',
--- a/mercurial/templatekw.py	Fri Dec 22 21:59:38 2017 +0900
+++ b/mercurial/templatekw.py	Sun Feb 25 15:43:35 2018 +0900
@@ -139,16 +139,16 @@
 def showdict(name, data, mapping, plural=None, key='key', value='value',
              fmt='%s=%s', separator=' '):
     c = [{key: k, value: v} for k, v in data.iteritems()]
-    f = _showlist(name, c, mapping, plural, separator)
+    f = _showlist(name, c, mapping['templ'], mapping, plural, separator)
     return hybriddict(data, key=key, value=value, fmt=fmt, gen=f)
 
 def showlist(name, values, mapping, plural=None, element=None, separator=' '):
     if not element:
         element = name
-    f = _showlist(name, values, mapping, plural, separator)
+    f = _showlist(name, values, mapping['templ'], mapping, plural, separator)
     return hybridlist(values, name=element, gen=f)
 
-def _showlist(name, values, mapping, plural=None, separator=' '):
+def _showlist(name, values, templ, mapping, plural=None, separator=' '):
     '''expand set of values.
     name is name of key in template map.
     values is list of strings or dicts.
@@ -169,7 +169,6 @@
 
     expand 'end_foos'.
     '''
-    templ = mapping['templ']
     strmapping = pycompat.strkwargs(mapping)
     if not plural:
         plural = name + 's'
@@ -383,7 +382,7 @@
     bookmarks = args['ctx'].bookmarks()
     active = repo._activebookmark
     makemap = lambda v: {'bookmark': v, 'active': active, 'current': active}
-    f = _showlist('bookmark', bookmarks, args)
+    f = _showlist('bookmark', bookmarks, args['templ'], args)
     return _hybrid(f, bookmarks, makemap, pycompat.identity)
 
 @templatekeyword('children')
@@ -455,7 +454,7 @@
     extras = util.sortdict((k, extras[k]) for k in sorted(extras))
     makemap = lambda k: {'key': k, 'value': extras[k]}
     c = [makemap(k) for k in extras]
-    f = _showlist('extra', c, args, plural='extras')
+    f = _showlist('extra', c, args['templ'], args, plural='extras')
     return _hybrid(f, extras, makemap,
                    lambda k: '%s=%s' % (k, util.escapestr(extras[k])))
 
@@ -589,7 +588,7 @@
     }
 
     tags = latesttags[2]
-    f = _showlist('latesttag', tags, args, separator=':')
+    f = _showlist('latesttag', tags, args['templ'], args, separator=':')
     return _hybrid(f, tags, makemap, pycompat.identity)
 
 @templatekeyword('latesttagdistance')
@@ -674,10 +673,10 @@
 
     for k, ns in repo.names.iteritems():
         names = ns.names(repo, ctx.node())
-        f = _showlist('name', names, args)
+        f = _showlist('name', names, args['templ'], args)
         namespaces[k] = _hybrid(f, names, makensmapfn(ns), pycompat.identity)
 
-    f = _showlist('namespace', list(namespaces), args)
+    f = _showlist('namespace', list(namespaces), args['templ'], args)
 
     def makemap(ns):
         return {
@@ -807,7 +806,8 @@
 
         data.append({'successors': successors, 'markers': finalmarkers})
 
-    f = _showlist('succsandmarkers', data, pycompat.byteskwargs(args))
+    args = pycompat.byteskwargs(args)
+    f = _showlist('succsandmarkers', data, args['templ'], args)
     return _hybrid(f, data, lambda x: x, pycompat.identity)
 
 @templatekeyword('p1rev', requires={'ctx'})
@@ -854,7 +854,7 @@
                 ('node', p.hex()),
                 ('phase', p.phasestr())]
                for p in pctxs]
-    f = _showlist('parent', parents, args)
+    f = _showlist('parent', parents, args['templ'], args)
     return _hybrid(f, prevs, lambda x: {'ctx': repo[x], 'revcache': {}},
                    lambda x: scmutil.formatchangeid(repo[x]), keytype=int)
 
@@ -881,7 +881,7 @@
     be evaluated"""
     args = pycompat.byteskwargs(args)
     repo = args['ctx'].repo()
-    f = _showlist(name, ['%d' % r for r in revs], args)
+    f = _showlist(name, ['%d' % r for r in revs], args['templ'], args)
     return _hybrid(f, revs,
                    lambda x: {name: x, 'ctx': repo[x], 'revcache': {}},
                    pycompat.identity, keytype=int)