Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date: Thu, 14 Feb 2019 16:24:03 -0800
From: Nick Bray <>
Subject: [PATCH] eliminate strict parentheses warnings for byteswaps

Patch is attached.

The change was motivated by including Musl's headers from a project
with -Werror -Wall enabled. I believe Fucshia ran into this, too.  I
couldn't think of a good regression test, in part because warnings are
somewhat compiler specific.  One possible approach would be to enable
-Werror -Wall in the main build, but that runs into issues of which
compilers are supported and what the core developers prefer.  Another
approach would be to only lint the header files - generate a dummy .c
file that includes all the header files and compile it with -Werror.
This is complicated by the "redirection" header files that warn you
should use the canonical version.  Which header files should be
checked?  So for the moment I punted on regression testing.  I mention
this line of thinking in case anyone has some perspective.

View attachment "0001-eliminate-strict-parentheses-warnings-for-byteswaps.patch" of type "text/x-patch" (1673 bytes)

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.