Skip to content
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

Break a retain cycle #444

Merged
merged 1 commit into from
Jul 5, 2013
Merged

Break a retain cycle #444

merged 1 commit into from
Jul 5, 2013

Conversation

hsoi
Copy link
Contributor

@hsoi hsoi commented Jul 5, 2013

Breaking a retain cycle that seems to be at the root of some table scrolling lockup problems. See #425

rs pushed a commit that referenced this pull request Jul 5, 2013
@rs rs merged commit 4246846 into SDWebImage:master Jul 5, 2013
@hsoi
Copy link
Contributor Author

hsoi commented Jul 13, 2013

thank you!

This was referenced Aug 8, 2013
rs pushed a commit that referenced this pull request Aug 9, 2013
RoCry pushed a commit to RoCry/SDWebImage that referenced this pull request Aug 14, 2013
hsoi pushed a commit to barz/SDWebImage that referenced this pull request Dec 20, 2013
…work, which you can trace via the github issues (e.g. SDWebImage#466) seem to have brought about better solutions. Somehow tho, my fix remained in our branch. I recently diff’d our ‘barz-master’ branch vs the ‘rs:master’ and while a few things are kosher to keep, this should go to maintain closer parity with the upstream/master.
devedup pushed a commit to FilmFlexMovies/SDWebImage that referenced this pull request Sep 10, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants