Fix issues with RenderTarget leaking memory.

not-fl3 / macroquad

Cross-platform game engine in Rust.

Apache License 2.0

3.04k stars 297 forks source link

Closed profan closed 2 months ago

profan commented 2 months ago

Fix issue with RenderTarget leaking memory by adding a drop impl, reported in issue #635

The memory leak issues just floating around were bothering me, and seemed simple enough to fix so figured why not :)

I also looked at #628 but couldn't actually reproduce the issue, so left that one alone for now.

profan commented 2 months ago

Let me split this PR into two parts so that the Atlas leak issue can be fixed separately of this bigger RenderTarget/RenderPass issue quickly

... There we go, split the PR so discussion for the render target bit can continue here if needed

not-fl3 commented 2 months ago

Thanks for PR!