Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

force event evaluation #2

Merged
merged 1 commit into from Jul 18, 2014

Conversation

Projects
None yet
2 participants
Contributor

gelisam commented Jul 18, 2014

I discovered a pretty serious memory leak which occurs when one kind of events is not used by the user's FRP expression. I am very new to reactive banana, so there might be easier ways to fix it, but this patch seems to work in all cases.

@gelisam gelisam force event evaluation
If the events are not used in the reactive-banana network, each event
will leak a bit of memory. This is especially bad with tick events,
which occur all the time, and to a lesser extend mouse movement events.
041765a

@Twey Twey added a commit that referenced this pull request Jul 18, 2014

@Twey Twey Merge pull request #2 from gelisam/leak
force event evaluation
9b20f87

@Twey Twey merged commit 9b20f87 into Twey:master Jul 18, 2014

Owner

Twey commented Jul 18, 2014

This doesn't seem like something we should have to do, but can't hurt. Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment