templater: do not preprocess template string in "if" expression (
issue4714)
The problem was spotted at
5ab28a2e9962, that says "this patch invokes it
with "strtoken='rawstring'" in "_evalifliteral()", because "t" is the result
of "arg" evaluation and it should be "string-escape"-ed if "arg" is "string"
expression." This workaround is no longer valid since
890845af1ac2 introduced
strict parsing of '\{'.
Instead, we should interpret bare token as "string" or "rawstring" template.
This is what buildmap() does at parsing phase.
--- a/mercurial/templater.py Fri Jun 05 12:57:21 2015 -0700
+++ b/mercurial/templater.py Mon Jun 08 18:14:22 2015 +0900
@@ -326,12 +326,13 @@
yield dictarg.get(key)
def _evalifliteral(arg, context, mapping):
- t = stringify(arg[0](context, mapping, arg[1]))
- if arg[0] == runstring or arg[0] == runrawstring:
+ # get back to token tag to reinterpret string as template
+ strtoken = {runstring: 'string', runrawstring: 'rawstring'}.get(arg[0])
+ if strtoken:
yield runtemplate(context, mapping,
- compiletemplate(t, context, strtoken='rawstring'))
+ compiletemplate(arg[1], context, strtoken))
else:
- yield t
+ yield stringify(arg[0](context, mapping, arg[1]))
def if_(context, mapping, args):
""":if(expr, then[, else]): Conditionally execute based on the result of
--- a/tests/test-command-template.t Fri Jun 05 12:57:21 2015 -0700
+++ b/tests/test-command-template.t Mon Jun 08 18:14:22 2015 +0900
@@ -2284,6 +2284,17 @@
\{rev} \{file} \\\head1
$ cd ..
+Test leading backslashes in "if" expression (issue4714):
+
+ $ cd latesttag
+ $ hg log -r 2 -T '{if("1", "\{rev}")} {if("1", r"\{rev}")}\n'
+ {rev} \2
+ $ hg log -r 2 -T '{if("1", "\\{rev}")} {if("1", r"\\{rev}")}\n'
+ \2 \\2
+ $ hg log -r 2 -T '{if("1", "\\\{rev}")} {if("1", r"\\\{rev}")}\n'
+ \{rev} \\\2
+ $ cd ..
+
"string-escape"-ed "\x5c\x786e" becomes r"\x6e" (once) or r"n" (twice)
$ hg log -R a -r 0 --template '{if("1", "\x5c\x786e", "NG")}\n'