X-Git-Url: http://git.tdb.fi/?p=libs%2Fgl.git;a=blobdiff_plain;f=source%2Ftexture.cpp;h=f1689145c8af9c5754569213a7561b0505e63b2c;hp=fb409e895c7fc3a0d221ee829648024462d76a55;hb=56beca9d8b4f7b4edac81411d31e24df88e84ac3;hpb=4cb33c263a00787a57531dfc41a4128bf34d7bee

diff --git a/source/texture.cpp b/source/texture.cpp
index fb409e89..f1689145 100644
--- a/source/texture.cpp
+++ b/source/texture.cpp
@@ -7,6 +7,7 @@
 #include <msp/gl/extensions/sgis_generate_mipmap.h>
 #include <msp/io/memory.h>
 #include <msp/strings/format.h>
+#include "bindable.h"
 #include "error.h"
 #include "resourcemanager.h"
 #include "resources.h"
@@ -63,11 +64,12 @@ Texture::Texture(GLenum t, ResourceManager *m):
 	ifmt(RGB),
 	min_filter(NEAREST_MIPMAP_LINEAR),
 	mag_filter(LINEAR),
+	mipmap_levels(0),
 	max_anisotropy(1.0f),
 	wrap_s(REPEAT),
 	wrap_t(REPEAT),
 	wrap_r(REPEAT),
-	gen_mipmap(false),
+	auto_gen_mipmap(0),
 	compare(false),
 	cmp_func(LEQUAL),
 	dirty_params(0)
@@ -156,7 +158,7 @@ void Texture::update_parameter(int mask) const
 	if(mask&WRAP_R)
 		set_parameter_i(GL_TEXTURE_WRAP_R, wrap_r);
 	if(mask&GENERATE_MIPMAP)
-		set_parameter_i(GL_GENERATE_MIPMAP, gen_mipmap);
+		set_parameter_i(GL_GENERATE_MIPMAP, auto_gen_mipmap!=0);
 	if(mask&COMPARE)
 		set_parameter_i(GL_TEXTURE_COMPARE_MODE, (compare ? GL_COMPARE_R_TO_TEXTURE : GL_NONE));
 	if(mask&COMPARE_FUNC)
@@ -178,6 +180,8 @@ void Texture::update_parameter(int mask) const
 				glTexParameteriv(target, GL_TEXTURE_SWIZZLE_RGBA, swizzle_orders+swizzle*4);
 		}
 	}
+	if(mask&MIPMAP_LEVELS)
+		set_parameter_i(GL_TEXTURE_MAX_LEVEL, (mipmap_levels ? mipmap_levels-1 : 1000));
 }
 
 void Texture::set_parameter_i(GLenum param, int value) const
@@ -214,6 +218,12 @@ void Texture::set_filter(TextureFilter f)
 	set_mag_filter(f==NEAREST ? NEAREST : LINEAR);
 }
 
+void Texture::set_mipmap_levels(unsigned l)
+{
+	mipmap_levels = l;
+	update_parameter(MIPMAP_LEVELS);
+}
+
 void Texture::set_max_anisotropy(float a)
 {
 	if(a<1.0f)
@@ -257,24 +267,31 @@ bool Texture::can_generate_mipmap()
 	return (EXT_framebuffer_object || SGIS_generate_mipmap);
 }
 
-void Texture::set_generate_mipmap(bool gm)
+void Texture::generate_mipmap()
 {
-	if(gm && !EXT_framebuffer_object)
-		static Require _req(SGIS_generate_mipmap);
-	gen_mipmap = gm;
-	if(!EXT_framebuffer_object)
-		update_parameter(GENERATE_MIPMAP);
+	// glGenerateMipmap is defined here
+	static Require _req(EXT_framebuffer_object);
+
+	if(ARB_direct_state_access)
+		glGenerateTextureMipmap(id);
+	else
+	{
+		BindRestore _bind(this);
+		glGenerateMipmap(target);
+	}
 }
 
-void Texture::auto_generate_mipmap()
+void Texture::set_auto_generate_mipmap(bool gm)
 {
-	// glGenerateMipmap is defined here
 	if(EXT_framebuffer_object)
+		auto_gen_mipmap = gm;
+	else
 	{
-		if(ARB_direct_state_access)
-			glGenerateTextureMipmap(id);
-		else
-			glGenerateMipmap(target);
+		if(gm)
+			static Require _req(SGIS_generate_mipmap);
+
+		auto_gen_mipmap = gm*2;
+		update_parameter(GENERATE_MIPMAP);
 	}
 }
 
@@ -301,7 +318,7 @@ void Texture::load_image(const string &fn, bool srgb)
 	image(img, srgb);
 }
 
-void Texture::bind_to(unsigned i) const
+void Texture::bind_to(unsigned i, bool legacy) const
 {
 	if(!id)
 	{
@@ -314,22 +331,30 @@ void Texture::bind_to(unsigned i) const
 		}
 	}
 
+	legacy = (legacy && is_legacy_target(target));
+
 	TexUnit &unit = TexUnit::get_unit(i);
 	const Texture *old = unit.get_texture();
-	if(unit.set_texture(this))
+	bool old_legacy = unit.get_texture_legacy();
+	if(unit.set_texture(this, legacy))
 	{
 		if(manager)
 			manager->resource_used(*this);
 
-		unit.bind();
-		if(unit.supports_legacy())
+		if(ARB_direct_state_access && !old_legacy && (!unit.supports_legacy() || !legacy))
+			glBindTextureUnit(i, id);
+		else
 		{
-			if(old && old->target!=target)
-				glDisable(old->target);
-			if(!old || old->target!=target)
-				glEnable(target);
+			unit.bind();
+			if(unit.supports_legacy())
+			{
+				if(old && old->target!=target && old_legacy)
+					glDisable(old->target);
+				if((!old || old->target!=target) && legacy)
+					glEnable(target);
+			}
+			glBindTexture(target, id);
 		}
-		glBindTexture(target, id);
 
 		if(dirty_params)
 		{
@@ -348,15 +373,26 @@ void Texture::unbind_from(unsigned i)
 {
 	TexUnit &unit = TexUnit::get_unit(i);
 	const Texture *cur = unit.get_texture();
+	bool legacy = unit.get_texture_legacy();
 	if(unit.set_texture(0))
 	{
-		unit.bind();
-		glBindTexture(cur->target, 0);
-		if(unit.supports_legacy())
-			glDisable(cur->target);
+		if(ARB_direct_state_access && !legacy)
+			glBindTextureUnit(i, 0);
+		else
+		{
+			unit.bind();
+			glBindTexture(cur->target, 0);
+			if(unit.supports_legacy() && legacy)
+				glDisable(cur->target);
+		}
 	}
 }
 
+bool Texture::is_legacy_target(GLenum target)
+{
+	return target<GL_TEXTURE_1D_ARRAY;
+}
+
 
 Texture::Loader::Loader(Texture &t):
 	DataFile::CollectionObjectLoader<Texture>(t, 0)
@@ -384,6 +420,7 @@ void Texture::Loader::init()
 	add("mag_filter", &Loader::mag_filter);
 	add("max_anisotropy", &Loader::max_anisotropy);
 	add("min_filter", &Loader::min_filter);
+	add("mipmap_levels", &Loader::mipmap_levels);
 	add("wrap",       &Loader::wrap);
 	add("wrap_r",     &Loader::wrap_r);
 	add("wrap_s",     &Loader::wrap_s);
@@ -394,6 +431,8 @@ void Texture::Loader::external_image(const string &fn)
 {
 	Graphics::Image img;
 	RefPtr<IO::Seekable> io = get_collection().open_raw(fn);
+	if(!io)
+		throw IO::file_not_found(fn);
 	img.load_io(*io);
 
 	obj.image(img, srgb);
@@ -406,7 +445,7 @@ void Texture::Loader::filter(TextureFilter f)
 
 void Texture::Loader::generate_mipmap(bool gm)
 {
-	obj.set_generate_mipmap(gm);
+	obj.set_auto_generate_mipmap(gm);
 }
 
 void Texture::Loader::image_data(const string &data)
@@ -433,6 +472,11 @@ void Texture::Loader::min_filter(TextureFilter f)
 	obj.set_min_filter(f);
 }
 
+void Texture::Loader::mipmap_levels(unsigned l)
+{
+	obj.set_mipmap_levels(l);
+}
+
 void Texture::Loader::wrap(TextureWrap w)
 {
 	obj.set_wrap(w);