[mono-vb] Poor state of mb-parser.jay
Miguel de Icaza
miguel@ximian.com
Mon, 21 Mar 2005 12:46:41 -0500
Hey,
> It seems the rules added to support the LIKE operator in your latest
> commit are giving recursive problems, as jay tell us:
>
> make[1]: Entering directory `/home/rafael/source/cli/mcs/mbas'
> ../jay/jay -ctv <../jay/skeleton.cs mb-parser.jay >mb-parser.cs
> ../jay/jay: 358 shift/reduce conflicts, 154 reduce/reduce conflicts.
>
> reduce/reduce conflicts normally make lots of false errors appear
> while parsing, but it doesn't show up in my limited testing
> (Test/misc).
This is correct; A reduce/reduce error typically represents a serious
problem, and they always introduce ambiguity in the grammar. Each one
of those must be very carefully reviewed.
I would advocate reverting that patch until that is fixed.
Miguel.