bison-patches
[Top][All Lists]
Advanced

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

Re: remove YYFAIL from lalr1.java and yacc.c


From: Paolo Bonzini
Subject: Re: remove YYFAIL from lalr1.java and yacc.c
Date: Mon, 21 Dec 2009 09:53:52 +0100
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.5) Gecko/20091209 Fedora/3.0-4.fc12 Lightning/1.0pre Thunderbird/3.0


, and I think that
breaking YYFAIL now for %error-verbose is fine too.

Google code search turns up an instance where both are used:

   
http://www.google.com/codesearch?hl=en&lr=&q=%25error-verbose+YYFAIL+file%3A\.y&sbtn=Search

Hmm, right. They're not relying on the semantics, though and they can simply fix it by removing the "|| defined (YYFAIL)". So it's effectively a false positive of the warning. Well, let's live with it.

I agree we may have to keep it for a long time, but hopefully the warning
will prompt users to explore at least why YYFAIL is deprecated.  Maybe I
should make the warning even stronger by saying that YYFAIL will
eventually be removed.

Fine, even if both of us know how likely that is to happen. :-)

Paolo




reply via email to

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