[cfe-commits] r65241 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.def lib/Sema/SemaType.cpp test/SemaObjC/protocol-archane.m

[Chris Lattner]

On Feb 23, 2009, at 6:33 AM, Steve Naroff wrote:

>
> On Feb 23, 2009, at 1:11 AM, Chris Lattner wrote:
>
>>
>> On Feb 21, 2009, at 11:50 AM, Steve Naroff wrote:
>>
>>> Author: snaroff
>>> Date: Sat Feb 21 13:50:43 2009
>>> New Revision: 65241
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=65241&view=rev
>>> Log:
>>> Warn about bogus protocol qualifiers.
>>
>> Should this be a hard error?
>>
>
> I made it a warning since GCC doesn't diagnose it at all.

Sounds like a bug in GCC.

> I'd be happier with an error,

Lets do it, it doesn't make any sense and it is trivial for people to  
"fix" their code (just remove protocols).

-Chris