[cfe-commits] [patch] Make it so that some warnings in DiagGroup<"all"> aren't on by default
Nico Weber
thakis at chromium.org
Tue Feb 14 18:28:19 PST 2012
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
More information about the cfe-commits
mailing list