r203506 - Turn on the uncovered umbrella header warning by default
Ben Langmuir
blangmuir at apple.com
Mon Mar 10 15:34:24 PDT 2014
Author: benlangmuir
Date: Mon Mar 10 17:34:23 2014
New Revision: 203506
URL: http://llvm.org/viewvc/llvm-project?rev=203506&view=rev
Log:
Turn on the uncovered umbrella header warning by default
This warning (under -Wincomplete-umbrella) diagnoses cases that are
difficult to understand without the compiler's help, since the symptom
is likely to be that you are missing the contents of headers that are
mistakenly omitted from a module. This seems like a good thing to have
on by default (with the usual caveat for -Wsystem-headers).
Modified:
cfe/trunk/include/clang/Basic/DiagnosticLexKinds.td
Modified: cfe/trunk/include/clang/Basic/DiagnosticLexKinds.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticLexKinds.td?rev=203506&r1=203505&r2=203506&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/DiagnosticLexKinds.td (original)
+++ cfe/trunk/include/clang/Basic/DiagnosticLexKinds.td Mon Mar 10 17:34:23 2014
@@ -606,7 +606,7 @@ def warn_auto_module_import : Warning<
"import of module '%1'">, InGroup<AutoImport>, DefaultIgnore;
def warn_uncovered_module_header : Warning<
"umbrella header for module '%0' does not include header '%1'">,
- InGroup<IncompleteUmbrella>, DefaultIgnore;
+ InGroup<IncompleteUmbrella>;
def warn_forgotten_module_header : Warning<
"header '%0' is included in module '%1' but not listed in module map">,
InGroup<IncompleteModule>, DefaultIgnore;
More information about the cfe-commits
mailing list