search

dyne / frei0r

A large collection of free and portable video plugins
https://frei0r.dyne.org/
GNU General Public License v2.0
419 stars 91 forks source link

Filmgrain - fix memory leak when blur = 0 #193

Closed esmane closed 1 month ago

esmane commented 1 month ago

I also moved the framebuffer allocation to the f0r_update function (so it should be thread safe now) and I swapped out my clamp function for the one in frei0r._math.h even though I doubt there's that much performance difference with that.

jaromil commented 1 month ago

to trigger a minor release at least one commit should start with 4 chars fix: including colon (semantic release auto versioning). If you think you are done with fixes, I will rename the commit messages like that and merge this too. Thanks!

esmane commented 1 month ago

Okay, thanks, I will do that in the future. Let me look it over a bit today and see if there's anything else I want to fix, then I'll let you know and I'll be done for a while and you can merge it.

esmane commented 1 month ago

Did some more fixes. I think I'm finally good, so if it looks good to you too please go ahead and send it. Thanks!