Skip to content

fixed memory leak#725

Open
roths wants to merge 1 commit intogoatslacker:masterfrom
roths:master
Open

fixed memory leak#725
roths wants to merge 1 commit intogoatslacker:masterfrom
roths:master

Conversation

@roths
Copy link

@roths roths commented May 9, 2018

No description provided.

@CrazyPython
Copy link

@goatslacker could you retry the Travis build? It appears to be failing because of something unrelated to the PR.

@matsjoyce
Copy link

To completely fix this issue you need to patch the function returned from Store.listen as well. +1 though, this was causing a huge memory leak in my app.

@jacaetevha
Copy link

This fix is really needed for our project. Is there any chance it will get merged, or do we need to just fork and apply the PR to our own fork? I'd rather not have to do that.

@matsjoyce
Copy link

This PR has been sitting here for over a year already, so I wouldn't wait for it to be merged.

@davidosomething
Copy link

i made the fix to the Store.listen in #736

Repository owner deleted a comment from chasedson Feb 7, 2024
Repository owner deleted a comment from diego-gutlop Feb 23, 2024
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

Successfully merging this pull request may close these issues.

5 participants