Mercurial > public > mercurial-scm > hg-stable
diff mercurial/templater.py @ 37274:7d3bc1d4e871
templater: pass (context, mapping) down to unwraphybrid()
See the subsequent patches for why.
I initially thought it would be wrong to pass a mapping to flatten() and
stringify() since these functions may be applied to a tree of generators,
where each node should be bound to the mapping when it was evaluated. But,
actually that isn't a problem. If an intermediate node has to override a
mapping dict, it can do on unwraphybrid() and yield "unwrapped" generator
of byte strings:
"{f(g(v))}" # literal template example.
^^^^ # g() want to override a mapping, so it returns a wrapped
# object 'G{V}' with partial mapping 'lm' attached.
^^^^^^^ # f() stringifies 'G{V}', starting from a mapping 'm'.
# when unwrapping 'G{}', it updates 'm' with 'lm', and
# passes it to 'V'.
This structure is important for the formatter (and the hgweb) to build a
static template keyword, which can't access a mapping dict until evaluation
phase.
author | Yuya Nishihara <yuya@tcha.org> |
---|---|
date | Sat, 17 Mar 2018 20:09:05 +0900 |
parents | b56b79185aad |
children | 3235afdfcf1c |
line wrap: on
line diff
--- a/mercurial/templater.py Mon Apr 02 16:18:33 2018 -0700 +++ b/mercurial/templater.py Sat Mar 17 20:09:05 2018 +0900 @@ -679,7 +679,7 @@ if extramapping: extramapping.update(mapping) mapping = extramapping - return templateutil.flatten(func(self, mapping, data)) + return templateutil.flatten(self, mapping, func(self, mapping, data)) engines = {'default': engine}