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

Kill Your Giant viewDidLoad #14

Open
acicartagena opened this issue Apr 4, 2016 · 1 comment
Open

Kill Your Giant viewDidLoad #14

acicartagena opened this issue Apr 4, 2016 · 1 comment

Comments

@acicartagena
Copy link

Just a question on the last part of the article, in using self inside the lazy property initialisation.

I'm not sure, but wouldn't that introduce a memory cycle, and rather use [weak self] in. Or is that not needed?

Thank you :)

@nickoneill
Copy link
Owner

Good question @angela-! It seems logical to me that the closure is used once and discarded (removing any self capture) but I don't know for sure. I will try to profile later and find out

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

No branches or pull requests

2 participants