Add a hack so that RGBA pixels are flipped properly in unit tests.

[movit] / test_util.cpp

diff --git a/test_util.cpp b/test_util.cpp
index 1f7dc1fe8629a2191df3f35123cffb1892875be9..cb9263b2fc0f8c680fb3926b061f64e276dcea2a 100644 (file)
--- a/test_util.cpp
+++ b/test_util.cpp
@@ -34,6 +34,14 @@ EffectChainTester::EffectChainTester(const float *data, unsigned width, unsigned
        check_error();
 }
 
+EffectChainTester::~EffectChainTester()
+{
+       glDeleteFramebuffers(1, &fbo);
+       check_error();
+       glDeleteTextures(1, &texnum);
+       check_error();
+}
+
 Input *EffectChainTester::add_input(const float *data, MovitPixelFormat pixel_format, ColorSpace color_space, GammaCurve gamma_curve)
 {
        ImageFormat format;
@@ -59,6 +67,10 @@ void EffectChainTester::run(float *out_data, GLenum format, ColorSpace color_spa
        glBindFramebuffer(GL_FRAMEBUFFER, fbo);
        glReadPixels(0, 0, width, height, format, GL_FLOAT, out_data);
 
+       if (format == GL_RGBA) {
+               width *= 4;
+       }
+
        // Flip upside-down to compensate for different origin.
        for (unsigned y = 0; y < height / 2; ++y) {
                unsigned flip_y = height - y - 1;