bison-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Dubious features


From: Akim Demaille
Subject: Re: Dubious features
Date: Thu, 8 Jun 2006 22:13:45 +0200


Le 8 juin 06 à 18:43, Paul Eggert a écrit :

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.

I'm sorry, but I don't follow.  This is not a feature, it is
just non-sense.  I fail to see the point in trying to turn it into
something perfect.  The bug I'm emulating is the feature.

Are there any other Yaccs presenting this feature?  There has been
many Bison which did not have it, and no one ever complained.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]