[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: RFC: explicit empty right-hand side of a rule
From: |
Joel E. Denny |
Subject: |
Re: RFC: explicit empty right-hand side of a rule |
Date: |
Mon, 4 Feb 2013 22:03:34 -0500 (EST) |
User-agent: |
Alpine 2.00 (DEB 1167 2008-08-23) |
Hi Akim,
On Mon, 4 Feb 2013, Akim Demaille wrote:
> I feel that using %empty should be all or nothing, so any occurrence
> or %empty should enable -Wempty-rule, right?
Makes sense. Would -Wno-empty-rule still disable it?
> > Also, in gcc and clang, -Wall does not include the default warnings.
> > It's a separate category. Quite a misnomer. Maybe we should just not
> > have a -Wall.
>
> We already have one. I have tried to model Bison's diagnostic interface
> to the one of gcc/clang. In this regard, it would be weird not to support
> -Wall, which is fairly traditional.
I misunderstood your proposal when you mentioned -Weverything. I realize
now you meant that -Wempty-rule would be included in -Weverything but not
in -Wall because the latter might be in widespread use. Right?
If we really want -Wall to work like gcc's, then should -Wno-all also
behave like gcc's? That is, perhaps it shouldn't disable default
warnings?
- Re: RFC: explicit empty right-hand side of a rule, Joel E. Denny, 2013/02/03
- Re: RFC: explicit empty right-hand side of a rule, Akim Demaille, 2013/02/03
- Re: RFC: explicit empty right-hand side of a rule, Joel E. Denny, 2013/02/03
- Re: RFC: explicit empty right-hand side of a rule, Akim Demaille, 2013/02/04
- Re: RFC: explicit empty right-hand side of a rule,
Joel E. Denny <=
- Re: RFC: explicit empty right-hand side of a rule, Akim Demaille, 2013/02/05
- Re: RFC: explicit empty right-hand side of a rule, Joel E. Denny, 2013/02/05
- [PATCH 00/12] {master} explicit empty right-hand side of a rule, Akim Demaille, 2013/02/18
- Re: [PATCH 00/12] {master} explicit empty right-hand side of a rule, Akim Demaille, 2013/02/16
- Re: [PATCH 00/12] {master} explicit empty right-hand side of a rule, Akim Demaille, 2013/02/19
- [PATCH 02/12] parser: use %empty, Akim Demaille, 2013/02/18
- [PATCH 01/12] grammar: introduce %empty, Akim Demaille, 2013/02/18
- Re: [PATCH 01/12] grammar: introduce %empty, Akim Demaille, 2013/02/18
- [PATCH 10/12] style: no longer use backquotes in messages, Akim Demaille, 2013/02/18
- [PATCH 08/12] doc: introduce %empty and -Wempty-rule, Akim Demaille, 2013/02/18