Re: [PATCH 0/3] {master} convert the conflict messages to a more bison-like form

[Paul Eggert]

On 09/24/2012 12:40 AM, Akim Demaille wrote:
> #1
>   foo.y: warning: 1 shift/reduce conflict [-Wconflicts-sr]
>   foo.y: warning: 2 reduce/reduce conflicts [-Wconflicts-rr]
> 
>   bar.y: 1 shift/reduce conflict, expected 0
>   bar.y: 2 reduce/reduce conflicts, expected 0
> 
> Proposal #2:
>   foo.y: warning: shift/reduce conflicts: 1 found [-Wconflicts-sr]
>   foo.y: warning: reduce/reduce conflicts: 2 found [-Wconflicts-rr]
> 
>   bar.y: shift/reduce conflicts: 1 found, 2 expected
>   bar.y: reduce/reduce conflicts: 2 found, 0 expected

Ignoring the problem of implementation and just thinking
about nice English, I prefer #1 for the first set of warnings
and #2 for the second set.  That is:

   foo.y: warning: 1 shift/reduce conflict [-Wconflicts-sr]
   foo.y: warning: 2 reduce/reduce conflicts [-Wconflicts-rr]

   bar.y: shift/reduce conflicts: 1 found, 2 expected
   bar.y: reduce/reduce conflicts: 2 found, 0 expected

Prefixing the second set with "error:" would be fine.