Crash on second selection at one stack table controller

[YuriSarkisyan]

I'm found a possible bug in realization of CLCascadeView. It reproduced in iOS 4.3 and 5.1 only in release compilation.

If user select table view cell (CLExampleTableViewController) then, by default, a new controller will created and inserted in stack. But, when user select cell in this table again (the same or other no difference), app will crash.

I'm use instruments and zombie profiler and see what CLCascadeView _pages instance is released in method hitTest:withEvent:. I think the reason is wrong return from _pages enumeration. Solution is to not return from inside of while cycle, but outside.

This is my version of this method:

- (UIView *)hitTest:(CGPoint)point withEvent:(UIEvent *)event {

id item = nil;
// create enumerator
NSEnumerator* enumerator = [_pages reverseObjectEnumerator];
// enumarate pages
UIView *view = nil;

while ((item = [enumerator nextObject])) {
    if (item != [NSNull null]) {

        UIView* page = (UIView*)item;
        CGRect rect = [_scrollView convertRect:page.frame toView:self];

        if (CGRectContainsPoint(rect, point)) {
            CGPoint newPoint = [self convertPoint:point toView:page];
            view = [page hitTest:newPoint withEvent:event];
        }
    }
} 
if (view != nil) {
    return view;
}

return [super hitTest:point withEvent:event];
}

[emilwojtaszek]

thank you! now it works ;p

[YuriSarkisyan]

One small addition, after assigning a value to a variable view inside the loop, you need to add break; Otherwise not properly defined view in landscape orientation.

[emilwojtaszek]

done :)

[jazuara]

You should remove the break; command, cause make it crash, but thanks for the upper solution!