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
