New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

notificationDelegate is declared as a strong reference property which causes the delegate to not be deallocated #24

Open

apiejh opened this issue Aug 28, 2016 · 0 comments

apiejh commented Aug 28, 2016

There is a memory leak issue, the propery should be declared as:
`@property (nonatomic, weak) id notificationDelegate;

Without the above both the delegate and the notification instance are caught in a retention cycle and neither is deallocated leading to memory leaks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment