-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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.
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
Possible memory leak in FBRequest? #10
Comments
This is a very good point. Will address in the next fix. Thanks! |
just updated, thanks for your comment |
5 tasks
5 tasks
5 tasks
This issue was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When you allocate the NSURLConnection in FBRequest.connect (line 271) - you do the following:
_connection = [[[NSURLConnection alloc] initWithRequest:request delegate:self] retain];
But if you call "alloc" you do not need an extra "retain".
I removed that "retain" and it still works.
The text was updated successfully, but these errors were encountered: