[cfe-commits] r76494 - in /cfe/trunk: include/clang/Basic/DiagnosticParseKinds.td test/SemaCXX/destructor.cpp

[Fariborz Jahanian]

Author: fjahanian
Date: Mon Jul 20 17:41:12 2009
New Revision: 76494

URL: http://llvm.org/viewvc/llvm-project?rev=76494&view=rev
Log:
Improve message for bad destructor decl. Per Doug's comment.


Modified:
    cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td
    cfe/trunk/test/SemaCXX/destructor.cpp

Modified: cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td?rev=76494&r1=76493&r2=76494&view=diff

==============================================================================
--- cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td (original)
+++ cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td Mon Jul 20 17:41:12 2009
@@ -139,7 +139,7 @@
   "unexpected type name %0: expected identifier">;
 def err_expected_class_name : Error<"expected class name">;
 def err_destructor_class_name : Error<
-  "destructor name must be same as the class name">;
+  "expected the class name after '~' to name a destructor">;
 def err_unspecified_vla_size_with_static : Error<
   "'static' may not be used with an unspecified variable length array size">;
 

Modified: cfe/trunk/test/SemaCXX/destructor.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/destructor.cpp?rev=76494&r1=76493&r2=76494&view=diff

==============================================================================
--- cfe/trunk/test/SemaCXX/destructor.cpp (original)
+++ cfe/trunk/test/SemaCXX/destructor.cpp Mon Jul 20 17:41:12 2009
@@ -40,8 +40,9 @@
   ~F(); // expected-error {{destructor cannot be redeclared}}
 };
 
-~; // expected-error {{destructor name must be same as the class name}}
-~undef(); // expected-error {{destructor name must be same as the class name}}
+~; // expected-error {{expected the class name after '~' to name a destructor}}
+~undef(); // expected-error {{expected the class name after '~' to name a destructor}}
+~operator+(int, int);  // expected-error {{expected the class name after '~' to name a destructor}}
 ~F(){} // expected-error {{destructor must be a non-static member function}}
 
 struct G {