X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fcore%2Fvertexsetup.cpp;h=da79e7f6135f9b3d06badf30e024903e7d5b280a;hb=d40673bd28c4b4524d3642b949d9d109dc6f9f24;hp=6a41397a8135a63931cffaec766d73f55a032476;hpb=ea7832c7c1ffab00cc1168bc8c41375fdd0eae86;p=libs%2Fgl.git

diff --git a/source/core/vertexsetup.cpp b/source/core/vertexsetup.cpp
index 6a41397a..da79e7f6 100644
--- a/source/core/vertexsetup.cpp
+++ b/source/core/vertexsetup.cpp
@@ -5,9 +5,11 @@
 #include <msp/gl/extensions/arb_vertex_attrib_binding.h>
 #include <msp/gl/extensions/arb_vertex_buffer_object.h>
 #include <msp/gl/extensions/arb_vertex_shader.h>
+#include <msp/gl/extensions/khr_debug.h>
 #include "buffer.h"
 #include "error.h"
 #include "gl.h"
+#include "misc.h"
 #include "vertexarray.h"
 #include "vertexsetup.h"
 
@@ -36,29 +38,53 @@ VertexSetup::~VertexSetup()
 	glDeleteVertexArrays(1, &id);
 }
 
+void VertexSetup::set_format(const VertexFormat &vfmt)
+{
+	if(!verify_format(vfmt))
+		throw invalid_argument("VertexSetup::set_format");
+	if(!vertex_format.empty())
+		throw invalid_operation("VertexSetup::set_format");
+
+	vertex_format = vfmt;
+}
+
+void VertexSetup::set_format_instanced(const VertexFormat &vfmt, const VertexFormat &ifmt)
+{
+	if(!verify_format(vfmt) || !verify_format(ifmt))
+		throw invalid_argument("VertexSetup::set_format");
+	if(!vertex_format.empty())
+		throw invalid_operation("VertexSetup::set_format");
+
+	vertex_format = vfmt;
+	inst_format = ifmt;
+}
+
 void VertexSetup::set_vertex_array(const VertexArray &a)
 {
+	if(vertex_format.empty())
+		throw invalid_operation("VertexSetup::set_vertex_array");
+	if(a.get_format()!=vertex_format)
+		throw incompatible_data("VertexSetup::set_vertex_array");
 	if(!a.get_buffer())
 		throw invalid_argument("VertexSetup::set_vertex_array");
 
 	vertex_array = &a;
-	update(get_update_mask(VERTEX_ARRAY, vertex_format, *vertex_array));
-	vertex_format = vertex_array->get_format();
+	update(VERTEX_ARRAY);
 }
 
-void VertexSetup::set_instance_array(const VertexArray *a)
+void VertexSetup::set_instance_array(const VertexArray &a)
 {
-	if(a)
-	{
-		if(!a->get_buffer())
-			throw invalid_argument("VertexSetup::set_instance_array");
+	if(inst_format.empty())
+		throw invalid_operation("VertexSetup::set_instance_array");
+	if(a.get_format()!=inst_format)
+		throw incompatible_data("VertexSetup::set_instance_array");
+	if(!a.get_buffer())
+		throw invalid_argument("VertexSetup::set_instance_array");
 
-		static Require req(ARB_instanced_arrays);
-	}
+	static Require req(ARB_instanced_arrays);
 
-	inst_array = a;
-	update(get_update_mask(INSTANCE_ARRAY, inst_format, *inst_array));
-	inst_format = inst_array->get_format();
+	inst_array = &a;
+	update(INSTANCE_ARRAY);
 }
 
 void VertexSetup::set_index_buffer(const Buffer &ibuf)
@@ -67,32 +93,20 @@ void VertexSetup::set_index_buffer(const Buffer &ibuf)
 	update(INDEX_BUFFER);
 }
 
-void VertexSetup::refresh()
+bool VertexSetup::verify_format(const VertexFormat &fmt)
 {
-	if(vertex_array && vertex_array->get_format()!=vertex_format)
-		set_vertex_array(*vertex_array);
+	if(fmt.empty())
+		return false;
 
-	if(inst_array && inst_array->get_format()!=inst_format)
-		set_instance_array(inst_array);
-}
+	static int max_attribs = -1;
+	if(max_attribs<0)
+		max_attribs = get_i(GL_MAX_VERTEX_ATTRIBS);
 
-unsigned VertexSetup::get_attribs(const VertexFormat &fmt)
-{
-	unsigned mask = 0;
 	for(const unsigned char *a=fmt.begin(); a!=fmt.end(); ++a)
-	{
-		unsigned sem = get_attribute_semantic(*a);
-		if(sem>=get_attribute_semantic(ATTRIB1))
-			sem -= get_attribute_semantic(ATTRIB1);
-		mask |= 1<<sem;
-	}
-	return mask;
-}
+		if(static_cast<int>(get_attribute_semantic(*a))>=max_attribs)
+			return false;
 
-unsigned VertexSetup::get_update_mask(unsigned base, const VertexFormat &cur_fmt, const VertexArray &new_array)
-{
-	unsigned unused = get_attribs(cur_fmt)&~get_attribs(new_array.get_format());
-	return base | (unused ? UNUSED_ATTRIBS | (unused<<ATTRIB_SHIFT) : 0);
+	return true;
 }
 
 void VertexSetup::update(unsigned mask) const
@@ -104,18 +118,6 @@ void VertexSetup::update(unsigned mask) const
 		return;
 	}
 
-	if(mask&UNUSED_ATTRIBS)
-	{
-		for(unsigned i=0, am=mask>>ATTRIB_SHIFT; am; ++i, am>>=1)
-			if(am&1)
-			{
-				if(direct)
-					glDisableVertexArrayAttrib(id, i);
-				else
-					glDisableVertexAttribArray(i);
-			}
-	}
-
 	if(mask&VERTEX_ARRAY)
 		update_vertex_array(*vertex_array, 0, 0, direct);
 
@@ -147,8 +149,6 @@ void VertexSetup::update_vertex_array(const VertexArray &array, unsigned binding
 	for(const unsigned char *a=fmt.begin(); a!=fmt.end(); ++a)
 	{
 		unsigned sem = get_attribute_semantic(*a);
-		if(sem>=get_attribute_semantic(ATTRIB1))
-			sem -= get_attribute_semantic(ATTRIB1);
 		unsigned sz = get_attribute_size(*a);
 		if(direct)
 		{
@@ -198,5 +198,51 @@ void VertexSetup::unbind()
 		glBindVertexArray(0);
 }
 
+void VertexSetup::unload()
+{
+	if(ARB_direct_state_access)
+	{
+		glVertexArrayVertexBuffer(id, 0, 0, 0, 0);
+		glVertexArrayVertexBuffer(id, 1, 0, 0, 0);
+		glVertexArrayElementBuffer(id, 0);
+	}
+	else
+	{
+		BindRestore _bind(*this);
+		Buffer::unbind_from(ARRAY_BUFFER);
+
+		for(const unsigned char *a=vertex_format.begin(); a!=vertex_format.end(); ++a)
+		{
+			unsigned sem = get_attribute_semantic(*a);
+			glDisableVertexAttribArray(sem);
+			glVertexAttribPointer(sem, 1, GL_FLOAT, false, 0, 0);
+		}
+		for(const unsigned char *a=inst_format.begin(); a!=inst_format.end(); ++a)
+		{
+			unsigned sem = get_attribute_semantic(*a);
+			glDisableVertexAttribArray(sem);
+			glVertexAttribPointer(sem, 1, GL_FLOAT, false, 0, 0);
+		}
+
+		glBindBuffer(ELEMENT_ARRAY_BUFFER, 0);
+	}
+
+	vertex_array = 0;
+	vertex_format = VertexFormat();
+	inst_array = 0;
+	inst_format = VertexFormat();
+	index_buffer = 0;
+}
+
+void VertexSetup::set_debug_name(const string &name)
+{
+#ifdef DEBUG
+	if(KHR_debug)
+		glObjectLabel(GL_VERTEX_ARRAY, id, name.size(), name.c_str());
+#else
+	(void)name;
+#endif
+}
+
 } // namespace GL
 } // namespace Msp