Re: Dubious features

[Paul Eggert]

Akim Demaille <address@hidden> writes:

>       * src/scan-gram.l: No longer "parse" things after `%union' until
>       `{'.  Rather, implement the simpler behavior of Solaris Yacc:
>       match a single token from '%union[^{]*'.

I don't like this, because it breaks a grammar that has a comment
containing "{" between the %union and the following brace.  E.g.:

  %union /* There is a special case for "{" in this union.  */
     {
       /* Strings that do not contain "{".  */
       char const *brace_free_strings;

       /* Strings that contain "{".  */
       char *braced_strings;
     }

Solaris yacc is clearly broken in this regard, but we shouldn't
emulate the bug.