What is the motivation for this patch? Clang defaulting to more warnings on than gcc is a feature, not a bug.
-Chris On 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 > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
