search

rwbarton / rw

DCSS bot
BSD 3-Clause "New" or "Revised" License
4 stars 2 forks source link

fix memory leak #1

Open rwbarton opened 11 years ago

rwbarton commented 11 years ago

Something in the reactive-banana network is leaking memory.

The leak goes away when `R.union fmap (const " ") goodbyeis removed fromclearText`, so maybe it has to do with steppers that are updated very rarely.

rwbarton commented 11 years ago

I have a workaround for now but it would be good to understand the leak. 567c54910fd43cc19eb84b5e748d82382e4dc2be is the leaky version.

rwbarton commented 11 years ago

Isolated the leak to this minimal example: http://hpaste.org/78029

rwbarton commented 11 years ago

The reactive-banana space leak is fixed in a development version of reactive-banana that's not on Hackage yet. Probably I have my own space leaks also.

rwbarton commented 10 years ago

That version is now on Hackage, but now I have this problem instead: https://github.com/HeinrichApfelmus/reactive-banana/issues/52#issuecomment-26686817

The plan is to write a wrapper module around Reactive.Banana that puts all Behavior values in Boxes and thereby controls evaluation explicitly.