[PATCH] parser: fix missing param in _match

Matt Mackall mpm at selenic.com
Tue Jun 8 16:00:01 CDT 2010


On Tue, 2010-06-08 at 17:28 +0200, Peter Arrenbrecht wrote:
> On Tue, Jun 8, 2010 at 4:28 PM, Peter Arrenbrecht
> <peter.arrenbrecht at gmail.com> wrote:
> > # HG changeset patch
> > # User Peter Arrenbrecht <peter.arrenbrecht at gmail.com>
> > # Date 1276007279 -7200
> > # Branch revset-dashes
> > # Node ID a7d8f5cbbf0c058905b306ff5f03acacb3cd8d83
> > # Parent  4a28385d026980bd0c671b09511dcab63e4badff
> > parser: fix missing param in _match
> 
> Any idea as to how I can provoke _match() to actually follow the error
> path? Tried to write a test, but no success.

It's not possible with the revset grammar, as '(' is the only piece that
uses matches and it's also the highest precedence so only ')' can end an
expression. But it might be possible with other grammars we'd add - I'm
exploring extending the templater grammar for instance.

-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list