Memory Leaks

[Kevin-Fanning]

Memory Leaks

[Kevin-Fanning]

I found another one in the 'remove_style_tags' function. this.$style_tags.remove() removes the selection from the dom, but it does not remove them from the $style_tags object. The styles are regenerated and added to this array, but the old ones are still there. These old ones stay and memory and won't be garbage collected. Setting $style_tags to $() removes the old references.

this.$style_tags.remove();
this.$style_tags = $();

[dsmorse]

Wow.. Great information. Do you have code fixes you could submit as pull requests?

[Kevin-Fanning]

I fixed these at work, so I won't be able to share that code. I can redo it at home when I get some time though.

[Nedlinin]

👍 @KFanPrecalcMan : Would love to see these changes implemented.