Now with many percent more patch attached.

On Tue, Feb 14, 2012 at 6:28 PM, Nico Weber <[email protected]> wrote:
> Hi,
>
> -Wall is defined like this:
>
>  def : DiagGroup<"all", [Most, Parentheses, Switch]>;
>
> This makes sense, but _all_ of the warnings in Parentheses and Switch
> are on by default, so -Wall is currently identical to -Wmost. With
> this patch, some of the warnings in that group become DefaultIgnore,
> so that they are only active if -Wall is specified. It looks like that
> was the intention for most of them, but since DiagGroup<"all"> is a
> bit confusing in this regard, they got always-enabled by accident (as
> far as I can tell).
>
> Ok?
>
> Nico

Attachment: clang-wall.patch
Description: Binary data

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to