X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Ftexture.cpp;h=f0c55999250263f641e546341e22b7545177748b;hb=9087cc0;hp=7983cfb50e5c4272f76c2ceb9e1c3a5628639635;hpb=78840e775c62c1b6e3f550bf8be1f0b828df91aa;p=libs%2Fgl.git

diff --git a/source/texture.cpp b/source/texture.cpp
index 7983cfb5..f0c55999 100644
--- 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);
 	}
 }