[cfe-commits] r108511 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td test/SemaCXX/nested-name-spec.cpp

Chandler Carruth chandlerc at gmail.com
Thu Jul 15 22:46:45 PDT 2010


Author: chandlerc
Date: Fri Jul 16 00:46:45 2010
New Revision: 108511

URL: http://llvm.org/viewvc/llvm-project?rev=108511&view=rev
Log:
Make this grossness default to the error it should always be.

Modified:
    cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td
    cfe/trunk/test/SemaCXX/nested-name-spec.cpp

Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td?rev=108511&r1=108510&r2=108511&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td (original)
+++ cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td Fri Jul 16 00:46:45 2010
@@ -2050,7 +2050,7 @@
   "parameter declarator cannot be qualified">;
 def ext_out_of_line_declaration : ExtWarn<
   "out-of-line declaration of a member must be a definition">,
-  InGroup<OutOfLineDeclaration>;
+  InGroup<OutOfLineDeclaration>, DefaultError;
 def note_member_def_close_match : Note<"member declaration nearly matches">;
 def err_typecheck_ivar_variable_size : Error<
   "instance variables must have a constant size">;

Modified: cfe/trunk/test/SemaCXX/nested-name-spec.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/nested-name-spec.cpp?rev=108511&r1=108510&r2=108511&view=diff
==============================================================================
--- cfe/trunk/test/SemaCXX/nested-name-spec.cpp (original)
+++ cfe/trunk/test/SemaCXX/nested-name-spec.cpp Fri Jul 16 00:46:45 2010
@@ -102,7 +102,7 @@
 
 int A2::RC::x; // expected-error{{non-static data member defined out-of-line}}
 
-void A2::CC::NC::m(); // expected-warning{{out-of-line declaration of a member must be a definition}}
+void A2::CC::NC::m(); // expected-error{{out-of-line declaration of a member must be a definition}}
 
 
 namespace E {





More information about the cfe-commits mailing list