Memory leak in headless `draw` function about react-native-skia HOT 3 CLOSED

Never mind! This is fixable by calling image.dispose() (and probably surface.dispose()).

I've opened a PR here.

Let me know if you want me to do anything else to support (e.g. write tests!)

from react-native-skia.

Thanks for the bug report and PR. We were always very aware of these potential issues and I think we just wanted to check first if there is appetite for the headless feature first.
Please give us some time to consider the best API there. We are looking at the using keyword: https://www.typescriptlang.org/docs/handbook/release-notes/typescript-5-2.html#using-declarations-and-explicit-resource-management, more specifically if it can be used on JSI host objects

from react-native-skia.

Yes! I think something like using makes a ton of sense, so that you can ensure cleanup.

No rush on this from my end! Feel free to close the Issue and the PR. I've already updated my code to call dispose on image and surface and I am monitoring it in production now!

from react-native-skia.