Separate abstract pixel compositions from concrete pixel formats

[libs/gl.git] / source / texture.cpp

diff --git a/source/texture.cpp b/source/texture.cpp
index 7983cfb50e5c4272f76c2ceb9e1c3a5628639635..f0c55999250263f641e546341e22b7545177748b 100644 (file)
--- a/source/texture.cpp
+++ b/source/texture.cpp
@@ -24,9 +24,9 @@ int Texture::swizzle_orders[] =
 Texture::Texture(GLenum t, ResourceManager *m):
        id(0),
        target(t),
-       ifmt(RGB),
+       ifmt(RGB8),
+       swizzle(NO_SWIZZLE),
        auto_gen_mipmap(false),
-       dirty_params(0),
        default_sampler(*this)
 {
        if(m)
@@ -46,82 +46,57 @@ Texture::~Texture()
                glDeleteTextures(1, &id);
 }
 
-DataType Texture::get_alloc_type(PixelFormat fmt)
-{
-       return (get_base_pixelformat(fmt)==DEPTH_COMPONENT ? UNSIGNED_SHORT : UNSIGNED_BYTE);
-}
-
 void Texture::set_internal_format(PixelFormat fmt)
 {
+       PixelComponents comp = get_components(fmt);
        FormatSwizzle swiz = NO_SWIZZLE;
-       if(ARB_texture_rg && ARB_texture_swizzle)
+       switch(comp)
        {
-               if(fmt==LUMINANCE)
-               {
-                       fmt = RED;
-                       swiz = R_TO_LUMINANCE;
-               }
-               else if(fmt==LUMINANCE_ALPHA)
-               {
-                       fmt = RG;
-                       swiz = RG_TO_LUMINANCE_ALPHA;
-               }
+       case LUMINANCE:
+               comp = RED;
+               swiz = R_TO_LUMINANCE;
+               break;
+       case LUMINANCE_ALPHA:
+               comp = RG;
+               swiz = RG_TO_LUMINANCE_ALPHA;
+               break;
+       default:;
        }
 
-       if(!get_component_size(fmt) && OES_required_internalformat)
-               fmt = get_default_sized_pixelformat(fmt);
-
+       fmt = make_pixelformat(comp, get_component_type(fmt));
        require_pixelformat(fmt);
+       if(swiz!=NO_SWIZZLE)
+               static Require _req(ARB_texture_swizzle);
        ifmt = fmt;
        swizzle = swiz;
-       if(swizzle)
-               update_parameter(FORMAT_SWIZZLE);
 }
 
-PixelFormat Texture::get_upload_format(PixelFormat fmt) const
+PixelComponents Texture::get_upload_components(PixelComponents comp) const
 {
-       if(fmt==LUMINANCE || fmt==LUMINANCE_ALPHA)
-               return get_base_pixelformat(ifmt);
+       if(comp==LUMINANCE || comp==LUMINANCE_ALPHA)
+               return get_components(ifmt);
        else
-               return fmt;
+               return comp;
 }
 
-void Texture::update_parameter(int mask) const
+void Texture::apply_swizzle()
 {
-       if(!id)
-       {
-               dirty_params |= mask;
+       if(swizzle==NO_SWIZZLE)
                return;
-       }
 
-       if(!ARB_direct_state_access && TexUnit::current().get_texture()!=this)
+       if(get_gl_api()==OPENGL_ES2)
        {
-               TexUnit *unit = TexUnit::find_unit(this);
-               if(!unit)
-               {
-                       dirty_params |= mask;
-                       return;
-               }
-
-               unit->bind();
+               set_parameter_i(GL_TEXTURE_SWIZZLE_R, swizzle_orders[swizzle*4]);
+               set_parameter_i(GL_TEXTURE_SWIZZLE_G, swizzle_orders[swizzle*4+1]);
+               set_parameter_i(GL_TEXTURE_SWIZZLE_B, swizzle_orders[swizzle*4+2]);
+               set_parameter_i(GL_TEXTURE_SWIZZLE_A, swizzle_orders[swizzle*4+3]);
        }
-
-       if(mask&FORMAT_SWIZZLE)
+       else
        {
-               if(get_gl_api()==OPENGL_ES2)
-               {
-                       set_parameter_i(GL_TEXTURE_SWIZZLE_R, swizzle_orders[swizzle*4]);
-                       set_parameter_i(GL_TEXTURE_SWIZZLE_G, swizzle_orders[swizzle*4+1]);
-                       set_parameter_i(GL_TEXTURE_SWIZZLE_B, swizzle_orders[swizzle*4+2]);
-                       set_parameter_i(GL_TEXTURE_SWIZZLE_A, swizzle_orders[swizzle*4+3]);
-               }
+               if(ARB_direct_state_access)
+                       glTextureParameteriv(id, GL_TEXTURE_SWIZZLE_RGBA, swizzle_orders+swizzle*4);
                else
-               {
-                       if(ARB_direct_state_access)
-                               glTextureParameteriv(id, GL_TEXTURE_SWIZZLE_RGBA, swizzle_orders+swizzle*4);
-                       else
-                               glTexParameteriv(target, GL_TEXTURE_SWIZZLE_RGBA, swizzle_orders+swizzle*4);
-               }
+                       glTexParameteriv(target, GL_TEXTURE_SWIZZLE_RGBA, swizzle_orders+swizzle*4);
        }
 }
 
@@ -133,14 +108,6 @@ void Texture::set_parameter_i(GLenum param, int value) const
                glTexParameteri(target, param, value);
 }
 
-void Texture::set_parameter_f(GLenum param, float value) const
-{
-       if(ARB_direct_state_access)
-               glTextureParameterf(id, param, value);
-       else
-               glTexParameterf(target, param, value);
-}
-
 void Texture::set_min_filter(TextureFilter f)
 {
        default_sampler.set_min_filter(f);
@@ -266,12 +233,6 @@ void Texture::bind_to(unsigned i) const
                        glBindTexture(target, id);
                }
 
-               if(dirty_params)
-               {
-                       update_parameter(dirty_params);
-                       dirty_params = 0;
-               }
-
                default_sampler.bind_to(i);
        }
 }