Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dynamic Mask: Implement sRGB and HDR support, and fix #819 #967

Merged
merged 1 commit into from
Nov 4, 2022
Merged

Dynamic Mask: Implement sRGB and HDR support, and fix #819 #967

merged 1 commit into from
Nov 4, 2022

Commits on Nov 4, 2022

  1. filter/dynamic-mask: Replace rendering code and optimize storage 

    The rendering code here was one of the older code bases, which was riddled with bugs and leaks. The new code doesn't look pretty, but it works for the time being until a better solution is found or made. It should be able to handle HDR inputs now, but it may not be completely correct yet. This also fixes the double-free bug.

As an additional improvement, I've moved the allocation of the effect to a shared class which should reduce the memory usage slightly when multiple effects are in play. And now selecting nothing selects the filter target itself without infinitely adding references to the filter. Good enough in my eyes.

Fixes #819
    @Xaymar
    Xaymar committed Nov 4, 2022
    Configuration menu
    Copy the full SHA
    48f57a3 View commit details
    Browse the repository at this point in the history