[cfe-commits] r65978 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.def lib/Sema/SemaStmt.cpp test/SemaObjC/catch-stmt.m

[Steve Naroff]

Author: snaroff
Date: Tue Mar  3 17:13:51 2009
New Revision: 65978

URL: http://llvm.org/viewvc/llvm-project?rev=65978&view=rev
Log:
Change a warning to an error...

Modified:
    cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.def
    cfe/trunk/lib/Sema/SemaStmt.cpp
    cfe/trunk/test/SemaObjC/catch-stmt.m

Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.def
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.def?rev=65978&r1=65977&r2=65978&view=diff

==============================================================================
--- cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.def (original)
+++ cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.def Tue Mar  3 17:13:51 2009
@@ -962,8 +962,8 @@
      "multiple garbage collection attributes specified for type")
 DIAG(err_catch_param_not_objc_type, ERROR,
      "@catch parameter is not an Objective-C class type")
-DIAG(warn_ignoring_qualifiers_on_catch_parm, WARNING,
-     "ignoring qualifiers on @catch parameter")
+DIAG(err_illegal_qualifiers_on_catch_parm, ERROR,
+     "illegal qualifiers on @catch parameter")
 
 
 // C++ casts

Modified: cfe/trunk/lib/Sema/SemaStmt.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaStmt.cpp?rev=65978&r1=65977&r2=65978&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/SemaStmt.cpp (original)
+++ cfe/trunk/lib/Sema/SemaStmt.cpp Tue Mar  3 17:13:51 2009
@@ -974,7 +974,7 @@
                        diag::err_catch_param_not_objc_type));
     if (PVD->getType()->isObjCQualifiedIdType())
       return StmtError(Diag(PVD->getLocation(), 
-                       diag::warn_ignoring_qualifiers_on_catch_parm));
+                       diag::err_illegal_qualifiers_on_catch_parm));
   }
     
   ObjCAtCatchStmt *CS = new (Context) ObjCAtCatchStmt(AtLoc, RParen,

Modified: cfe/trunk/test/SemaObjC/catch-stmt.m
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaObjC/catch-stmt.m?rev=65978&r1=65977&r2=65978&view=diff

==============================================================================
--- cfe/trunk/test/SemaObjC/catch-stmt.m (original)
+++ cfe/trunk/test/SemaObjC/catch-stmt.m Tue Mar  3 17:13:51 2009
@@ -7,7 +7,7 @@
   } @catch (void a) { // expected-error{{@catch parameter is not an Objective-C class type}}
   } @catch (int) { // expected-error{{@catch parameter is not an Objective-C class type}}
   } @catch (int *b) { // expected-error{{@catch parameter is not an Objective-C class type}}
-  } @catch (id <P> c) { // expected-warning{{ignoring qualifiers on @catch parameter}}
+  } @catch (id <P> c) { // expected-error{{illegal qualifiers on @catch parameter}}
   }
 }