On Mar 3, 2015, at 11:02 AM, jahanian <[email protected]> wrote:

> Patch looks good with couple of minors.
> Provide a note where receiver/argument has been declared.
> Place CheckObjCCircularContainer(Result) right after 
> checkRetainCycles(Result).

This patch does not address the general case of same expression used as 
receiver and addObject argument.
Is this something that you care enough to address? Need not be in this patch 
though.

> 
> - Fariborz
> 
> 

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to