[prev in list] [next in list] [prev in thread] [next in thread] 

List:       boost
Subject:    Re: [boost] Showstopper for Boost 1.56?
From:       Vladimir Prus <ghost () cs ! msu ! su>
Date:       2014-07-30 4:23:57
Message-ID: lr9s0t$ov8$1 () ger ! gmane ! org
[Download RAW message or body]

On 07/30/2014 04:58 AM, Niall Douglas wrote:
> On 29 Jul 2014 at 20:48, Edward Diener wrote:
>
>>> I think we all agree that 23Mb of warnings spew per build using Boost
>>> 1.56 with clang is unacceptable for release. Either
>>> Boost.Preprocessor needs fixing, or clang toolset builds explicitly
>>> need to add -Wno-c99-extensions to every Boost build.
>>
>> The change that was made in Preprocessor is that variadic macros support
>> is turned on for clang by default. I made this change after a number of
>> complaints from users compiling with clang and after verifying that
>> clang has always had variadic macro support.
>>
>> I do realize that clang will spit out lots of warnings, so maybe making
>> this change was not a good idea on a practical level.
>
> That's great news, now we know the cause of the spew!
>
> This is definitely a showstopper for Boost 1.56, megabytes of warning
> spew on clang would be embarrassing for an official release. Bug is
> filed at:
>
> https://svn.boost.org/trac/boost/ticket/10276

I suppose this is unfortunate all round - in particular using -Wno-c99-extensions is way
wider than the warning we're trying to suppress here.

In either case, for a simple change like this, a pull request might be better than
an issue.

- Volodya



_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
[prev in list] [next in list] [prev in thread] [next in thread]