Skip to content

Loading…

add super dealloc call #12

Closed
wants to merge 1 commit into from

4 participants

@gpascale

No description provided.

@Bo98

No, don't do this. This is an ARC project and [super dealloc] is for non-ARC projects.

If your app doesn't use ARC yet, then simply add the -fobjc-arc compiler flag for TSMiniWebBrowser files.

@tonisalae
Owner

@Bo98 is right. TSMiniWebBrowser uses ARC. So you can't call [super dealloc]

@alexcoplan

@tonisalae perhaps don't implement dealloc then - could you unhook the delegate somewhere else - then avoid using a mm method altogether?

@tonisalae tonisalae closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Sep 25, 2012
  1. @gpascale

    add super dealloc call

    gpascale committed
Showing with 1 addition and 0 deletions.
  1. +1 −0 TSMiniWebBrowser/TSMiniWebBrowser.m
View
1 TSMiniWebBrowser/TSMiniWebBrowser.m
@@ -90,6 +90,7 @@ -(void) dismissController {
-(void)dealloc
{
[webView setDelegate:nil];
+ [super dealloc];
}
#pragma mark - Init
Something went wrong with that request. Please try again.