Weak references in blocks and retain cycles

Question

In this question, I asked about the following code and retain cycles:

__weak Cell *weakSelf = self;
NSBlockOperation *op = [NSBlockOperation blockOperationWi         


        

Answer 1

Your old code creates this retain cycle if you don't use __weak:

• (NSBlockOperation *)op retains the outer block
• The outer block retains self (if you're not using __weak)
• self retains (NSOperationQueue *)renderQueue
• (NSOperationQueue *)renderQueue retains (NSBlockOperation *)op

None of the objects in that cycle can be deallocated unless one of those links is broken. But the code you showed us does break the retain cycle. When op finishes executing, renderQueue releases it, breaking the retain cycle.

I suspect that your new code creates this retain cycle:

• (MBItem *)close retains the block
• The block retains self
• self retains childObject
• childObject retains (NSMutableArray *)items
• (NSMutableArray *)items retains (MBItem *)close

If nothing happens to break one of those links, none of the objects in the cycle can be deallocated. You haven't shown us any code that breaks the retain cycle. If there is no event that explicitly breaks it (for example by clearing out childObject.items), then you need to use __weak to break the retain cycle.