Re: parenthesis warnings

[Evan Lavelle]

> On 17 Dec 2010, at 17:08, Gergely Feldhoffer wrote:
>
> > Thank you, we upgraded bison from latest debian 2.4.1 to 2.4.3, and it
> > works just fine.

I just had this problem on the default Ubuntu 10.10 installation (gcc 
4.4.5, bison 2.4.1). Anyone who wants to stay on 2.4.1 needs to edit the 
location.cc skeleton file to add the extra parentheses. In my case, this 
file is in /usr/share/bison. The source for the code at position.hh:136 
is at line 120 in location.cc; just add parentheses around the 2 
expressions on either side of the '||'.

I was surprised to see this in my grammars, though, since I don't use 
%locations. Should the position/location code be generated if it's not 
required?

-Evan