Fix a bug where we would send too few parameters down to the refocus shader.

[movit] / main.cpp

diff --git a/main.cpp b/main.cpp
index d9a116d56425c710b3ac06f9425faae7b3dd2ce6..3c807a1b7af588fb9b9cffb4a56f734b350d5f75 100644 (file)
--- a/main.cpp
+++ b/main.cpp
@@ -47,9 +47,9 @@ void update_hsv(Effect *lift_gamma_gain_effect, Effect *saturation_effect)
        RGBTriplet gamma(1.0f, 1.0f, 1.0f);
        RGBTriplet gain(1.0f, 1.0f, 1.0f);
 
-       hsv2rgb(lift_theta, lift_rad, lift_v, &lift.r, &lift.g, &lift.b);
-       hsv2rgb(gamma_theta, gamma_rad, gamma_v * 2.0f, &gamma.r, &gamma.g, &gamma.b);
-       hsv2rgb(gain_theta, gain_rad, gain_v * 4.0f, &gain.r, &gain.g, &gain.b);
+       hsv2rgb_normalized(lift_theta, lift_rad, lift_v, &lift.r, &lift.g, &lift.b);
+       hsv2rgb_normalized(gamma_theta, gamma_rad, gamma_v * 2.0f, &gamma.r, &gamma.g, &gamma.b);
+       hsv2rgb_normalized(gain_theta, gain_rad, gain_v * 4.0f, &gain.r, &gain.g, &gain.b);
 
        bool ok = lift_gamma_gain_effect->set_vec3("lift", (float *)&lift);
        ok = ok && lift_gamma_gain_effect->set_vec3("gamma", (float *)&gamma);
@@ -163,11 +163,10 @@ int main(int argc, char **argv)
        EffectChain chain(WIDTH, HEIGHT);
 
        ImageFormat inout_format;
-       inout_format.pixel_format = FORMAT_BGRA;
        inout_format.color_space = COLORSPACE_sRGB;
        inout_format.gamma_curve = GAMMA_sRGB;
 
-       FlatInput *input = new FlatInput(inout_format, WIDTH, HEIGHT);
+       FlatInput *input = new FlatInput(inout_format, FORMAT_BGRA, img_w, img_h);
        chain.add_input(input);
        Effect *lift_gamma_gain_effect = chain.add_effect(new LiftGammaGainEffect());
        Effect *saturation_effect = chain.add_effect(new SaturationEffect());