Add support to the ResourcePool for 2D textures.

[movit] / flat_input.cpp

diff --git a/flat_input.cpp b/flat_input.cpp
index 40fb7333bcec17fec73683579e7c796925c5afe8..5323dac19ead8663060eaa023f51ea000da2a995 100644 (file)
--- a/flat_input.cpp
+++ b/flat_input.cpp
@@ -4,6 +4,7 @@
 
 #include "effect_util.h"
 #include "flat_input.h"
+#include "resource_pool.h"
 #include "util.h"
 
 FlatInput::FlatInput(ImageFormat image_format, MovitPixelFormat pixel_format, GLenum type, unsigned width, unsigned height)
@@ -28,13 +29,8 @@ FlatInput::FlatInput(ImageFormat image_format, MovitPixelFormat pixel_format, GL
 
 FlatInput::~FlatInput()
 {
-       if (pbo != 0) {
-               glDeleteBuffers(1, &pbo);
-               check_error();
-       }
        if (texture_num != 0) {
-               glDeleteTextures(1, &texture_num);
-               check_error();
+               resource_pool->release_2d_texture(texture_num);
        }
 }
 
@@ -43,7 +39,7 @@ void FlatInput::finalize()
        // Translate the input format to OpenGL's enums.
        GLenum internal_format;
        if (type == GL_FLOAT) {
-               internal_format = GL_RGBA16F_ARB;
+               internal_format = GL_RGBA32F_ARB;
        } else if (output_linear_gamma) {
                assert(type == GL_UNSIGNED_BYTE);
                internal_format = GL_SRGB8_ALPHA8;
@@ -53,55 +49,27 @@ void FlatInput::finalize()
        }
        if (pixel_format == FORMAT_RGB) {
                format = GL_RGB;
-               bytes_per_pixel = 3;
        } else if (pixel_format == FORMAT_RGBA_PREMULTIPLIED_ALPHA ||
                   pixel_format == FORMAT_RGBA_POSTMULTIPLIED_ALPHA) {
                format = GL_RGBA;
-               bytes_per_pixel = 4;
        } else if (pixel_format == FORMAT_BGR) {
                format = GL_BGR;
-               bytes_per_pixel = 3;
        } else if (pixel_format == FORMAT_BGRA_PREMULTIPLIED_ALPHA ||
                   pixel_format == FORMAT_BGRA_POSTMULTIPLIED_ALPHA) {
                format = GL_BGRA;
-               bytes_per_pixel = 4;
        } else if (pixel_format == FORMAT_GRAYSCALE) {
                format = GL_LUMINANCE;
-               bytes_per_pixel = 1;
        } else {
                assert(false);
        }
-       if (type == GL_FLOAT) {
-               bytes_per_pixel *= sizeof(float);
-       }
 
-       // Create PBO to hold the texture holding the input image, and then the texture itself.
-       glGenBuffers(1, &pbo);
-       check_error();
-       glBindBuffer(GL_PIXEL_UNPACK_BUFFER_ARB, pbo);
-       check_error();
-       glBufferData(GL_PIXEL_UNPACK_BUFFER_ARB, pitch * height * bytes_per_pixel, NULL, GL_STREAM_DRAW);
-       check_error();
-       glBindBuffer(GL_PIXEL_UNPACK_BUFFER_ARB, 0);
-       check_error();
-       
-       glGenTextures(1, &texture_num);
-       check_error();
+       // Create the texture itself.
+       texture_num = resource_pool->create_2d_texture(internal_format, width, height);
        glBindTexture(GL_TEXTURE_2D, texture_num);
        check_error();
-       glPixelStorei(GL_UNPACK_ALIGNMENT, 1);
-       check_error();
        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, needs_mipmaps ? GL_LINEAR_MIPMAP_NEAREST : GL_LINEAR);
        check_error();
-       glPixelStorei(GL_UNPACK_ROW_LENGTH, pitch);
-       check_error();
-       // Intel/Mesa seems to have a broken glGenerateMipmap() for non-FBO textures, so do it here
-       // instead of calling glGenerateMipmap().
-       glTexParameteri(GL_TEXTURE_2D, GL_GENERATE_MIPMAP, needs_mipmaps ? GL_TRUE : GL_FALSE);
-       check_error();
-       glTexImage2D(GL_TEXTURE_2D, 0, internal_format, width, height, 0, format, type, NULL);
-       check_error();
-       glPixelStorei(GL_UNPACK_ROW_LENGTH, 0);
+       glBindTexture(GL_TEXTURE_2D, 0);
        check_error();
 
        needs_update = true;
@@ -114,21 +82,23 @@ void FlatInput::set_gl_state(GLuint glsl_program_num, const std::string& prefix,
        check_error();
        glBindTexture(GL_TEXTURE_2D, texture_num);
        check_error();
+       glPixelStorei(GL_UNPACK_ROW_LENGTH, 0);
+       check_error();
 
        if (needs_update) {
-               // Copy the pixel data into the PBO.
+               // Re-upload the texture.
                glBindBuffer(GL_PIXEL_UNPACK_BUFFER_ARB, pbo);
                check_error();
-               void *mapped_pbo = glMapBufferARB(GL_PIXEL_UNPACK_BUFFER_ARB, GL_WRITE_ONLY);
-               memcpy(mapped_pbo, pixel_data, pitch * height * bytes_per_pixel);
-               glUnmapBufferARB(GL_PIXEL_UNPACK_BUFFER_ARB);
+               glPixelStorei(GL_UNPACK_ALIGNMENT, 1);
                check_error();
-
-               // Re-upload the texture from the PBO.
                glPixelStorei(GL_UNPACK_ROW_LENGTH, pitch);
                check_error();
-               glTexSubImage2D(GL_TEXTURE_2D, 0, 0, 0, width, height, format, type, BUFFER_OFFSET(0));
+               glTexSubImage2D(GL_TEXTURE_2D, 0, 0, 0, width, height, format, type, pixel_data);
                check_error();
+               if (needs_mipmaps) {
+                       glGenerateMipmap(GL_TEXTURE_2D);
+                       check_error();
+               }
                glPixelStorei(GL_UNPACK_ROW_LENGTH, 0);
                check_error();
                glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE);