[cfe-commits] r156825 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaDecl.cpp test/SemaObjC/no-ivar-in-interface-block.m

[Ted Kremenek]

On May 15, 2012, at 10:44 AM, jahanian <fjahanian at apple.com> wrote:

> 
> On May 15, 2012, at 10:20 AM, John McCall wrote:
> 
>> On May 15, 2012, at 9:33 AM, Fariborz Jahanian wrote:
>>> Author: fjahanian
>>> Date: Tue May 15 11:33:04 2012
>>> New Revision: 156825
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=156825&view=rev
>>> Log:
>>> objective-c nonfragile abi: discourage ivar declarations
>>> in @interface by issuing warning (off by default) under
>>> opt'ed in flag -Winterface-block-ivar. // rdar://10763173
>> 
>> The 'block' doesn't seem to be adding much.  -Winterface-ivars?
> 
> Makes sense. In r156831.
> - Fariborz

Not to bike shed here, but I'd prefer:

  -Wobjc-interface-ivars

just to be very clear this is for Objective-C.  We've been following this trend lately with other warnings.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120522/68a9b207/attachment.html>