templater: make sure expression is terminated by '}'

This can't be ensured by the tokenize() generator since the parser stop
consuming tokens once it reached an end of an operation.

Spotted by Matt Harbison.

mercurial/templater.py

@@ -184,6 +184,8 b' def _parsetemplate(tmpl, start, stop, qu'
             return parsed, n + 1
 
         parseres, pos = p.parse(tokenize(tmpl, n + 1, stop, '}'))
+        if not tmpl.endswith('}', n + 1, pos):
+            raise error.ParseError(_("invalid token"), pos)
         parsed.append(parseres)
 
     if quote:

tests/test-command-template.t

@@ -2756,6 +2756,15 b' Error on syntax:'
   $ hg log -T '{date'
   hg: parse error at 1: unterminated template expansion
   [255]
+  $ hg log -T '{date(}'
+  hg: parse error at 7: not a prefix: end
+  [255]
+  $ hg log -T '{date)}'
+  hg: parse error at 5: invalid token
+  [255]
+  $ hg log -T '{date date}'
+  hg: parse error at 6: invalid token
+  [255]
 
 Behind the scenes, this will throw TypeError