[PATCH] [analysis] Add test for missing -dealloc method in classes with an ivar

David Kilzer ddkilzer at kilzer.net
Fri Aug 22 10:30:43 PDT 2014


>>! In D5023#6, @jordan_rose wrote:
> Yup, this checker predates property auto-synthesis!
> 
> I'm actually not sure we //want// to have this particular check; from just the ivar, you can't tell its ownership convention. The properties are very valuable though.

This false positive is filed as [[ rdar://problem/6074390 | radar 6074390 ]].

> Also, while you're here, make sure we don't do any of this under ARC!

Will do.  I guess I need to change the test file to include multiple "RUN" commands like this one:
  test/SemaObjC/warn-missing-super.m

http://reviews.llvm.org/D5023






More information about the cfe-commits mailing list