Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160627172159.GH10893@brightrain.aerifal.cx>
Date: Mon, 27 Jun 2016 13:21:59 -0400
From: Rich Felker <dalias@...c.org>
To: musl@...ts.openwall.com
Subject: Re: Fix warnings when build with clang

On Mon, Jun 27, 2016 at 10:01:52AM -0700, weimingz@...eaurora.org wrote:
> On 2016-06-25 02:37, Szabolcs Nagy wrote:
> >* Zhao, Weiming <weimingz@...eaurora.org> [2016-06-24 17:18:23 -0700]:
> >>Clang gives warnings about the missing parentheses for bitwise ops.
> >>
> >>It's not functional but just code readability.
> >>
> >>musl-m3/include/endian.h:32:25: warning: '&' within '|'
> >>[-Wbitwise-op-parentheses]
> >>        return __x>>24 | __x>>8&0xff00 | __x<<8&0xff0000 | __x<<24;
> >>                       ~ ~~~~~~^~~~~~~
> >
> >if clang warns about system headers that's a clang bug.
> 
> The endian.h is a musl file.

Right, which is why it's a system header.

Rich

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.