[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.