JSON-Patch icon indicating copy to clipboard operation
JSON-Patch copied to clipboard
verified publisher icon Starcounter-Jack

Memory leak still continues

Open pnowak opened this issue 7 years ago • 2 comments

Hi.

When I added JSON-Patch to dependencies (so after yours fix https://github.com/Starcounter-Jack/JSON-Patch/pull/212/files) and I implemented the slowCheck removed by you to our dataObserver - https://github.com/handsontable/handsontable/pull/5575 - we noticed that the memory leak still continues. Unless I did something wrong.

pnowak avatar Nov 20 '18 11:11 pnowak

Hi!

and I implemented the slowCheck removed by you

Could you elaborate more on this? It's true that I maintain this lib but I don't remember removing this. Could you link the PR or the commit I (or someone) did this? You'd make it easier for me to help.

Thanks

alshakero avatar Nov 20 '18 11:11 alshakero

I thought about this PR - https://github.com/Starcounter-Jack/JSON-Patch/pull/104 We need this slowCheck, because we need observe changes, not only react to DOM events.

pnowak avatar Nov 20 '18 11:11 pnowak