Skip to content
This repository has been archived by the owner on Jan 4, 2023. It is now read-only.

Reference cycle in Listings VC/VM #514

Open
ashfurrow opened this issue Sep 28, 2015 · 1 comment
Open

Reference cycle in Listings VC/VM #514

ashfurrow opened this issue Sep 28, 2015 · 1 comment
Labels

Comments

@ashfurrow
Copy link
Contributor

Someone raised an excellent point on Twitter that the listings VC that creates the VM creates a subtle reference cycle while referring the VC's functions for presenting modals. Importantly, force-unwrapping self won't work. He also recommends a fix that looks super-nice.

This isn't a big deal, since there's only one instance of each class, and they last the entire lifetime of the app, but in most code this isn't the case, so we should fix it and then write about or at least update the blog post.

@ashfurrow
Copy link
Contributor Author

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant