[cfe-commits] r105011 - in /cfe/trunk: include/clang/Basic/DiagnosticParseKinds.td lib/Parse/ParseDecl.cpp test/CodeGen/typedef-func.c test/Sema/enum.c test/Sema/function-redecl.c

[Fariborz Jahanian]

On May 28, 2010, at 3:34 PM, Eli Friedman wrote:

> On Fri, May 28, 2010 at 3:23 PM, Fariborz Jahanian <fjahanian at apple.com 
> > wrote:
>> Author: fjahanian
>> Date: Fri May 28 17:23:22 2010
>> New Revision: 105011
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=105011&view=rev
>> Log:
>> Empty enum in c is now error to match gcc's behavior.
>> (radar 8040068).
>>
>>
>> Modified:
>>    cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td
>>    cfe/trunk/lib/Parse/ParseDecl.cpp
>>    cfe/trunk/test/CodeGen/typedef-func.c
>>    cfe/trunk/test/Sema/enum.c
>>    cfe/trunk/test/Sema/function-redecl.c
>>
>> Modified: cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td?rev=105011&r1=105010&r2=105011&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td (original)
>> +++ cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td Fri May  
>> 28 17:23:22 2010
>> @@ -35,6 +35,7 @@
>>
>>  def ext_empty_struct_union_enum : Extension<"use of empty %0  
>> extension">;
>>
>> +def error_empty_enum : Error<"use of empty union">;
>
> Umm, maybe change the text of the error message?  It's kind of
> confusing to complain about a nonexistent union :)

Oops. Thanks. Fixed in r105013.

- Fariborz