From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22899 invoked by alias); 5 Oct 2012 16:32:32 -0000 Received: (qmail 22811 invoked by uid 48); 5 Oct 2012 16:32:04 -0000 From: "paolo.carlini at oracle dot com" To: gcc-bugs@gcc.gnu.org Subject: [Bug c++/54194] misleading suggestion about arithmetic in operand of '|' Date: Fri, 05 Oct 2012 16:32:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: c++ X-Bugzilla-Keywords: diagnostic X-Bugzilla-Severity: minor X-Bugzilla-Who: paolo.carlini at oracle dot com X-Bugzilla-Status: ASSIGNED X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: paolo.carlini at oracle dot com X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org X-SW-Source: 2012-10/txt/msg00449.txt.bz2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54194 --- Comment #5 from Paolo Carlini 2012-10-05 16:32:02 UTC --- Sure. Since the patch simply using warning_at is trivial and anyway warn_about_parentheses is currently shared with the C front end, I propose to just do that for 4.8.0. I'm attaching what I have regtested. I think it would be a definite improvement. Tell me what you think. If you disagree, no problem, I can move to something else and completely delay this issue to a later time.