Re: Bison test case fixes uncovered by recent bug reports

[Akim Demaille]

>>> "Joel" == Joel E Denny <address@hidden> writes:

 > I disagree.  There's no reason for parameters to yyerror() and to yylex()
 > to intrude on the parser's namespace.  I've noticed that the bison
 > documentation is careful about this.

There are reaons: at some places the test suite is tailored to work
with or with-out %pure-parser.  It is already quite a cathedral as it
is so that I don't which to have in addition to change the name of
yylval and yylloc depending on whether they are global variables or
arguments.  To be consistent and to ease the sharing of routines in
the test suite, I chose to always use these names.

It is very sane not to promote this in the documentation, but IMHO,
it's inappropriate for the test suite.  It will always be time to
change the test suite when we change it.