Mark a already-done TODO as fixed.

[movit] / lift_gamma_gain_effect.frag

diff --git a/lift_gamma_gain_effect.frag b/lift_gamma_gain_effect.frag
index acd9d5162ca34c219bfa555feb191c1ea5a2cbd8..ceb2081c752591e71277b10af64e8e499f2f2cb1 100644 (file)
--- a/lift_gamma_gain_effect.frag
+++ b/lift_gamma_gain_effect.frag
@@ -1,14 +1,26 @@
+// Implicit uniforms:
+//
 // These are calculated in the host code to save some arithmetic.
-uniform vec3 PREFIX(gain_pow_inv_gamma);  // gain^(1/gamma).
-uniform vec3 PREFIX(inv_gamma_22);  // 2.2 / gamma.
+// uniform vec3 PREFIX(gain_pow_inv_gamma);  // gain^(1/gamma).
+// uniform vec3 PREFIX(inv_gamma_22);  // 2.2 / gamma.
 
 vec4 FUNCNAME(vec2 tc) {
        vec4 x = INPUT(tc);
 
+       x.rgb /= x.aaa;
+
+       // pow() of negative numbers is undefined, so clip out-of-gamut values.
+       x.rgb = max(x.rgb, 0.0);
+
        x.rgb = pow(x.rgb, vec3(1.0/2.2));
        x.rgb += PREFIX(lift) * (vec3(1) - x.rgb);
+
+       // Clip out-of-gamut values again.
+       x.rgb = max(x.rgb, 0.0);
+
        x.rgb = pow(x.rgb, PREFIX(inv_gamma_22));
        x.rgb *= PREFIX(gain_pow_inv_gamma);
+       x.rgb *= x.aaa;
 
        return x;
 }