In EffectChainTester, there is no point in making the FBO and texture persistent.

diff --git a/test_util.cpp b/test_util.cpp
index f822fcc442e32f60820f92046fd9c9c24c3a8055..199be456ca4da8839882a46c5fdf2ee0e39f167e 100644 (file)
--- a/test_util.cpp
+++ b/test_util.cpp
@@ -47,52 +47,17 @@ void vertical_flip(T *data, unsigned width, unsigned height)
 EffectChainTester::EffectChainTester(const float *data, unsigned width, unsigned height,
                                      MovitPixelFormat pixel_format, Colorspace color_space, GammaCurve gamma_curve,
                                      GLenum framebuffer_format)
-       : chain(width, height, get_static_pool()), width(width), height(height), output_added(false), finalized(false)
+       : chain(width, height, get_static_pool()), width(width), height(height), framebuffer_format(framebuffer_format), output_added(false), finalized(false)
 {
        CHECK(init_movit(".", MOVIT_DEBUG_OFF));
 
        if (data != NULL) {
                add_input(data, pixel_format, color_space, gamma_curve);
        }
-
-       GLuint type;
-       if (framebuffer_format == GL_RGBA8) {
-               type = GL_UNSIGNED_BYTE;
-       } else if (framebuffer_format == GL_RGBA16F || framebuffer_format == GL_RGBA32F) {
-               type = GL_FLOAT;
-       } else {
-               // Add more here as needed.
-               assert(false);
-       }
-
-       glGenTextures(1, &texnum);
-       check_error();
-       glBindTexture(GL_TEXTURE_2D, texnum);
-       check_error();
-       glTexImage2D(GL_TEXTURE_2D, 0, framebuffer_format, width, height, 0, GL_RGBA, type, NULL);
-       check_error();
-
-       glGenFramebuffers(1, &fbo);
-       check_error();
-       glBindFramebuffer(GL_FRAMEBUFFER, fbo);
-       check_error();
-       glFramebufferTexture2D(
-               GL_FRAMEBUFFER,
-               GL_COLOR_ATTACHMENT0,
-               GL_TEXTURE_2D,
-               texnum,
-               0);
-       check_error();
-       glBindFramebuffer(GL_FRAMEBUFFER, 0);
-       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, int input_width, int input_height)
@@ -150,6 +115,39 @@ void EffectChainTester::internal_run(T *out_data, GLenum internal_format, GLenum
                finalize_chain(color_space, gamma_curve, alpha_format);
        }
 
+       GLuint type;
+       if (framebuffer_format == GL_RGBA8) {
+               type = GL_UNSIGNED_BYTE;
+       } else if (framebuffer_format == GL_RGBA16F || framebuffer_format == GL_RGBA32F) {
+               type = GL_FLOAT;
+       } else {
+               // Add more here as needed.
+               assert(false);
+       }
+
+       GLuint fbo, texnum;
+
+       glGenTextures(1, &texnum);
+       check_error();
+       glBindTexture(GL_TEXTURE_2D, texnum);
+       check_error();
+       glTexImage2D(GL_TEXTURE_2D, 0, framebuffer_format, width, height, 0, GL_RGBA, type, NULL);
+       check_error();
+
+       glGenFramebuffers(1, &fbo);
+       check_error();
+       glBindFramebuffer(GL_FRAMEBUFFER, fbo);
+       check_error();
+       glFramebufferTexture2D(
+               GL_FRAMEBUFFER,
+               GL_COLOR_ATTACHMENT0,
+               GL_TEXTURE_2D,
+               texnum,
+               0);
+       check_error();
+       glBindFramebuffer(GL_FRAMEBUFFER, 0);
+       check_error();
+
        chain.render_to_fbo(fbo, width, height);
 
        glBindFramebuffer(GL_FRAMEBUFFER, fbo);
@@ -178,6 +176,11 @@ void EffectChainTester::internal_run(T *out_data, GLenum internal_format, GLenum
                check_error();
        }
 
+       glDeleteFramebuffers(1, &fbo);
+       check_error();
+       glDeleteTextures(1, &texnum);
+       check_error();
+
        if (format == GL_RGBA) {
                width *= 4;
        }

diff --git a/test_util.h b/test_util.h
index a37e8220c075c9d4ce698a87921b52f5f34a1aac..f786708e5a6172f527ff0dea36e542b14c700586 100644 (file)
--- a/test_util.h
+++ b/test_util.h
@@ -33,8 +33,8 @@ private:
        void internal_run(T *out_data, GLenum internal_format, GLenum format, Colorspace color_space, GammaCurve gamma_curve, OutputAlphaFormat alpha_format = OUTPUT_ALPHA_FORMAT_POSTMULTIPLIED);
 
        EffectChain chain;
-       GLuint fbo, texnum;
        unsigned width, height;
+       GLenum framebuffer_format;
        bool output_added;
        bool finalized;
 };