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

Tiny mistake which is leading to potentially big memory/performance hit #1

Open
jormungand opened this issue Aug 29, 2020 · 0 comments

Comments

@jormungand
Copy link

jormungand commented Aug 29, 2020

let binder = Binder<State>.init(self, binding: { node, state in

Every time you call .stream the new binder object will be created despite the associated one already exists.
Probably it's better to wrap the Binder.init logic into a closure to pass further

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

1 participant