Thanks Nico! On Feb 15, 2012, at 10:18 AM, Nico Weber wrote:
> Author: nico > Date: Wed Feb 15 12:18:51 2012 > New Revision: 150600 > > URL: http://llvm.org/viewvc/llvm-project?rev=150600&view=rev > Log: > Tweak comment above DiagGroup<"all">. > > Modified: > cfe/trunk/include/clang/Basic/DiagnosticGroups.td > > Modified: cfe/trunk/include/clang/Basic/DiagnosticGroups.td > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticGroups.td?rev=150600&r1=150599&r2=150600&view=diff > ============================================================================== > --- cfe/trunk/include/clang/Basic/DiagnosticGroups.td (original) > +++ cfe/trunk/include/clang/Basic/DiagnosticGroups.td Wed Feb 15 12:18:51 2012 > @@ -334,7 +334,9 @@ > // Thread Safety warnings > def ThreadSafety : DiagGroup<"thread-safety">; > > -// -Wall is -Wmost -Wparentheses -Wdangling-else -Wswitch > +// Note that putting warnings in -Wall will not disable them by default. If a > +// warning should be active _only_ when -Wall is passed in, mark it as > +// DefaultIgnore in addition to putting it here. > def : DiagGroup<"all", [Most, Parentheses, Switch]>; > > // Aliases. > > > _______________________________________________ > 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
