X-Git-Url: http://git.tdb.fi/?p=libs%2Fgl.git;a=blobdiff_plain;f=source%2Fvertexarray.cpp;h=0ebd4f0fef4e9855a650b53280ddfbac1ffbcd99;hp=6d6699020334655edf2f7a899d3636572ee6071e;hb=ef9ffbc287417520d223ce29dc6c2de110cac945;hpb=4443707c752ab8ee288f1c22be08cf82f27439d7

diff --git a/source/vertexarray.cpp b/source/vertexarray.cpp
index 6d669902..0ebd4f0f 100644
--- a/source/vertexarray.cpp
+++ b/source/vertexarray.cpp
@@ -1,54 +1,87 @@
-/* $Id$
-
-This file is part of libmspgl
-Copyright © 2007-2010  Mikko Rasa, Mikkosoft Productions
-Distributed under the LGPL
-*/
-
-#include "arb_vertex_program.h"
-#include "extension.h"
+#include <msp/gl/extensions/arb_multitexture.h>
+#include <msp/gl/extensions/arb_vertex_shader.h>
+#include <msp/gl/extensions/msp_legacy_features.h>
+#include "buffer.h"
+#include "error.h"
 #include "gl.h"
-#include "version_1_2.h"
+#include "mesh.h"
 #include "vertexarray.h"
-#include "vertexbuffer.h"
 
 using namespace std;
 
 namespace Msp {
 namespace GL {
 
-VertexArray::VertexArray(const VertexFormat &f):
-	vbuf(0),
-	own_vbuf(false)
+bool VertexArray::legacy_used = false;
+
+VertexArray::VertexArray(const VertexFormat &f)
 {
 	reset(f);
 }
 
 VertexArray::~VertexArray()
 {
-	if(own_vbuf)
-		delete vbuf;
+	/* Unbind accesses the current VertexArray, so a call from ~Bindable would
+	try to access destroyed data. */
+	if(current()==this)
+		unbind();
 }
 
-void VertexArray::use_vertex_buffer()
+void VertexArray::reset(const VertexFormat &f)
 {
-	if(vbuf && own_vbuf)
-		return;
+	clear();
+	format = f;
+	stride = get_stride(format);
+	legacy = false;
+
+	arrays.clear();
+
+	unsigned offs = 0;
+	for(const unsigned char *c=format.begin(); c!=format.end(); ++c)
+	{
+		unsigned slot = get_array_slot(*c);
+		if(slot>=arrays.size())
+			arrays.resize(slot+1);
+
+		Array &arr = arrays[slot];
+		arr.component = *c;
+		arr.offset = offs;
 
-	vbuf = new Buffer(ARRAY_BUFFER);
-	own_vbuf = true;
+		if(*c<ATTRIB1)
+			legacy = true;
 
-	update_data();
+		offs += get_component_size(*c);
+	}
 }
 
-void VertexArray::use_vertex_buffer(Buffer *b)
+unsigned VertexArray::get_array_slot(unsigned char comp)
 {
-	if(own_vbuf)
-		delete vbuf;
-	vbuf = b;
-	own_vbuf = false;
+	unsigned t = get_component_type(comp);
+	if(t==get_component_type(VERTEX3))
+		return 0;
+	else if(t==get_component_type(NORMAL3))
+		return 1;
+	else if(t==get_component_type(COLOR4_FLOAT))
+		return 2;
+	else if(comp>=TEXCOORD1 && comp<=TEXCOORD4+12)
+	{
+		t -= get_component_type(TEXCOORD1);
+		if(t>0)
+			static Require _req(ARB_multitexture);
+		return 3+t;
+	}
+	else
+	{
+		static Require _req(ARB_vertex_shader);
+		if(comp>=ATTRIB1)
+			t -= get_component_type(ATTRIB1);
+		return 7+t;
+	}
+}
 
-	update_data();
+void VertexArray::clear()
+{
+	data.clear();
 }
 
 void VertexArray::reserve(unsigned n)
@@ -56,163 +89,190 @@ void VertexArray::reserve(unsigned n)
 	data.reserve(n*stride);
 }
 
-void VertexArray::clear()
+float *VertexArray::append()
 {
-	data.clear();
+	data.insert(data.end(), stride, 0.0f);
+	update_offset();
+	dirty = true;
+	return &*(data.end()-stride);
 }
 
-void VertexArray::reset(const VertexFormat &f)
+float *VertexArray::modify(unsigned i)
 {
-	clear();
-	format = f;
-	stride = get_stride(format);
+	dirty = true;
+	return &data[0]+i*stride;
+}
 
-	bool has_multitex = false;
-	bool has_gen_attrs = false;
-	for(const unsigned char *c=format.begin(); c!=format.end(); ++c)
-	{
-		if(*c>=TEXCOORD1+4 && *c<ATTRIB1)
-			has_multitex = true;
-		if(*c>=ATTRIB1)
-			has_gen_attrs = true;
-	}
-	if(has_multitex)
-		static RequireVersion _ver(1, 3);
-	if(has_gen_attrs)
-		static RequireExtension _ext("GL_ARB_vertex_program");
+unsigned VertexArray::get_data_size() const
+{
+	return data.size()*sizeof(float);
 }
 
-void VertexArray::apply() const
+void VertexArray::apply(bool use_legacy) const
 {
 	if(format.empty())
-		throw InvalidState("Trying to apply a vertex array with no data");
+		throw invalid_operation("VertexArray::apply");
+	// Don't mess up the vertex array object of a mesh
+	if(Mesh::current())
+		throw invalid_operation("VertexArray::apply");
+
+	/* Unbind first if the legacy flag changes.  The logic for supporting it
+	directly in apply_arrays would get too complicated, and this also allows
+	rebinding the same array with different legacy setting. */
+	if(legacy_used!=use_legacy)
+		unbind();
+
+	if(!use_legacy)
+		static Require _req(ARB_vertex_shader);
+	else if(legacy)
+		static Require _req(MSP_legacy_features);
+
+	const VertexArray *old = current();
+	/* If the array has been modified, apply it even if it was the last one to
+	be applied.  This is necessary to get the data updated to vertex buffer, and
+	to resync things after a format change.  Radeon drivers also have some
+	problems with modifying vertex arrays without re-setting the pointers. */
+	if(!set_current(this) && !dirty)
+		return;
 
-	if(vbuf)
-		vbuf->bind();
+	Buffer *vbuf = get_buffer();
+	Bind _bind_vbuf(vbuf, ARRAY_BUFFER);
+	if(vbuf && dirty)
+		update_buffer();
+
+	const float *base = (vbuf ? reinterpret_cast<float *>(get_offset()) : &data[0]);
+	unsigned stride_bytes = stride*sizeof(float);
+	apply_arrays(&arrays, (old ? &old->arrays : 0), base, stride_bytes, use_legacy);
+}
 
-	const float *base = vbuf?0:&data[0];
-	unsigned offset = 0;
-	ArrayMask found;
-	unsigned bpv = stride*sizeof(float);
+void VertexArray::apply_arrays(const vector<Array> *arrays, const vector<Array> *old_arrays, const float *base, unsigned stride_bytes, bool use_legacy)
+{
 	unsigned active_tex = 0;
-	for(const unsigned char *c=format.begin(); c!=format.end(); ++c)
+	unsigned n_arrays = arrays ? arrays->size() : 0;
+	if(old_arrays)
+		n_arrays = max<unsigned>(n_arrays, old_arrays->size());
+	for(unsigned i=0; i<n_arrays; ++i)
 	{
-		unsigned sz = (*c&3)+1;
-		unsigned t = *c>>2;
-		bool en = enabled_arrays.is_set(t);
-		switch(t)
+		const Array *arr = ((arrays && i<arrays->size() && (*arrays)[i].component) ? &(*arrays)[i] : 0);
+		const Array *old_arr = ((old_arrays && i<old_arrays->size() && (*old_arrays)[i].component) ? &(*old_arrays)[i] : 0);
+		if(!arr && !old_arr)
+			continue;
+
+		unsigned char comp = (arr ? arr->component : old_arr->component);
+		unsigned sz = get_component_size(comp);
+		unsigned t = get_component_type(comp);
+		if(use_legacy)
 		{
-		case 0:
-			glVertexPointer(sz, GL_FLOAT, bpv, base+offset);
-			if(!en)
-				glEnableClientState(GL_VERTEX_ARRAY);
-			break;
-		case 1:
-			glNormalPointer(GL_FLOAT, bpv, base+offset);
-			if(!en)
-				glEnableClientState(GL_NORMAL_ARRAY);
-			break;
-		case 2:
-			if(sz==1)
-				glColorPointer(4, GL_UNSIGNED_BYTE, bpv, base+offset);
-			else
-				glColorPointer(sz, GL_FLOAT, bpv, base+offset);
-			if(!en)
-				glEnableClientState(GL_COLOR_ARRAY);
-			break;
-		default:
-			if(t<11)
+			GLenum array_type = 0;
+			if(t==get_component_type(VERTEX3))
+			{
+				if(arr)
+					glVertexPointer(sz, GL_FLOAT, stride_bytes, base+arr->offset);
+				array_type = GL_VERTEX_ARRAY;
+			}
+			else if(t==get_component_type(NORMAL3))
 			{
-				if(t>3 || active_tex)
+				if(arr)
+					glNormalPointer(GL_FLOAT, stride_bytes, base+arr->offset);
+				array_type = GL_NORMAL_ARRAY;
+			}
+			else if(t==get_component_type(COLOR4_FLOAT))
+			{
+				if(arr)
 				{
-					glClientActiveTexture(GL_TEXTURE0+(t-3));
-					active_tex = t-3;
+					if(sz==1)
+						glColorPointer(4, GL_UNSIGNED_BYTE, stride_bytes, base+arr->offset);
+					else
+						glColorPointer(sz, GL_FLOAT, stride_bytes, base+arr->offset);
 				}
-				glTexCoordPointer(sz, GL_FLOAT, bpv, base+offset);
-				if(!en)
-					glEnableClientState(GL_TEXTURE_COORD_ARRAY);
+				array_type = GL_COLOR_ARRAY;
 			}
-			else
+			else if(comp>=TEXCOORD1 && comp<=TEXCOORD4+12)
 			{
-				glVertexAttribPointerARB(t-11, sz, GL_FLOAT, false, bpv, base+offset);
-				if(!en)
-					glEnableVertexAttribArrayARB(t-11);
+				t -= get_component_type(TEXCOORD1);
+				if(t>0 || active_tex)
+				{
+					glClientActiveTexture(GL_TEXTURE0+t);
+					active_tex = t;
+				}
+				if(arr)
+					glTexCoordPointer(sz, GL_FLOAT, stride_bytes, base+arr->offset);
+				array_type = GL_TEXTURE_COORD_ARRAY;
 			}
-			break;
-		}
-		found.set(t);
-		offset += sz;
-	}
 
-	for(unsigned i=0; i<64; ++i)
-		if(enabled_arrays.is_set(i) && !found.is_set(i))
-		{
-			if(i==0)
-				glDisableClientState(GL_VERTEX_ARRAY);
-			else if(i==1)
-				glDisableClientState(GL_NORMAL_ARRAY);
-			else if(i==2)
-				glDisableClientState(GL_COLOR_ARRAY);
-			else if(i>=3 && i<11)
+			if(array_type)
 			{
-				if(i>3 || active_tex)
-					glClientActiveTexture(GL_TEXTURE0+(i-3));
-				glDisableClientState(GL_TEXTURE_COORD_ARRAY);
+				// Only change enable state if needed
+				if(arr && !old_arr)
+					glEnableClientState(array_type);
+				else if(old_arr && !arr)
+					glDisableClientState(array_type);
+
+				continue;
 			}
+		}
+
+		if(t>=get_component_type(ATTRIB1))
+			t -= get_component_type(ATTRIB1);
+		if(arr)
+		{
+			if(arr->component==COLOR4_UBYTE)
+				glVertexAttribPointer(t, 4, GL_UNSIGNED_BYTE, true, stride_bytes, base+arr->offset);
 			else
-				glDisableVertexAttribArrayARB(i-11);
+				glVertexAttribPointer(t, sz, GL_FLOAT, false, stride_bytes, base+arr->offset);
 		}
 
-	enabled_arrays = found;
+		// Only change enable state if needed
+		if(arr && !old_arr)
+			glEnableVertexAttribArray(t);
+		else if(old_arr && !arr)
+			glDisableVertexAttribArray(t);
+	}
 
 	if(active_tex)
 		glClientActiveTexture(GL_TEXTURE0);
 
-	if(vbuf)
-		vbuf->unbind();
+	legacy_used = use_legacy;
 }
 
-/**
-Updates the VertexArray data to the VertexBuffer tied to the array, if any.
-*/
-void VertexArray::update_data()
+void VertexArray::unbind()
 {
-	if(vbuf)
-	{
-		vbuf->data(data.size()*sizeof(float), &data[0]);
-		vbuf->unbind();
-	}
+	const VertexArray *old = current();
+	if(set_current(0))
+		apply_arrays(0, &old->arrays, 0, 0, legacy_used);
 }
 
-float *VertexArray::append()
-{
-	data.insert(data.end(), stride, 0.0f);
-	return &*data.end()-stride;
-}
-
-VertexArray::ArrayMask VertexArray::enabled_arrays;
 
-
-VertexArray::ArrayMask::ArrayMask()
-{
-	for(unsigned i=0; i<N; ++i)
-		mask[i] = 0;
-}
-
-void VertexArray::ArrayMask::set(unsigned bit)
-{
-	mask[bit/B] |= 1<<(bit%B);
-}
-
-bool VertexArray::ArrayMask::is_set(unsigned bit) const
-{
-	return mask[bit/B]&(1<<(bit%B));
-}
+VertexArray::Array::Array():
+	component(0),
+	offset(0)
+{ }
 
 
 VertexArray::Loader::Loader(VertexArray &a):
 	VertexArrayBuilder(a)
 {
+	add("vertex", static_cast<void (Loader::*)(float, float)>(&Loader::vertex));
+	add("vertex", static_cast<void (Loader::*)(float, float, float)>(&Loader::vertex));
+	add("vertex", static_cast<void (Loader::*)(float, float, float, float)>(&Loader::vertex));
+	add("normal", static_cast<void (Loader::*)(float, float, float)>(&Loader::normal));
+	add("texcoord", static_cast<void (Loader::*)(float)>(&Loader::texcoord));
+	add("texcoord", static_cast<void (Loader::*)(float, float)>(&Loader::texcoord));
+	add("texcoord", static_cast<void (Loader::*)(float, float, float)>(&Loader::texcoord));
+	add("texcoord", static_cast<void (Loader::*)(float, float, float, float)>(&Loader::texcoord));
+	add("multitexcoord", static_cast<void (Loader::*)(unsigned, float)>(&Loader::multitexcoord));
+	add("multitexcoord", static_cast<void (Loader::*)(unsigned, float, float)>(&Loader::multitexcoord));
+	add("multitexcoord", static_cast<void (Loader::*)(unsigned, float, float, float)>(&Loader::multitexcoord));
+	add("multitexcoord", static_cast<void (Loader::*)(unsigned, float, float, float, float)>(&Loader::multitexcoord));
+	add("color", static_cast<void (Loader::*)(float, float, float)>(&Loader::color));
+	add("color", static_cast<void (Loader::*)(float, float, float, float)>(&Loader::color));
+	add("attrib", static_cast<void (Loader::*)(unsigned, float)>(&Loader::attrib));
+	add("attrib", static_cast<void (Loader::*)(unsigned, float, float)>(&Loader::attrib));
+	add("attrib", static_cast<void (Loader::*)(unsigned, float, float, float)>(&Loader::attrib));
+	add("attrib", static_cast<void (Loader::*)(unsigned, float, float, float, float)>(&Loader::attrib));
+	add("tangent", static_cast<void (Loader::*)(float, float, float)>(&Loader::tangent));
+	add("binormal", static_cast<void (Loader::*)(float, float, float)>(&Loader::binormal));
+
 	add("vertex2",   static_cast<void (Loader::*)(float, float)>(&Loader::vertex));
 	add("vertex3",   static_cast<void (Loader::*)(float, float, float)>(&Loader::vertex));
 	add("vertex4",   static_cast<void (Loader::*)(float, float, float, float)>(&Loader::vertex));
@@ -221,34 +281,18 @@ VertexArray::Loader::Loader(VertexArray &a):
 	add("texcoord2", static_cast<void (Loader::*)(float, float)>(&Loader::texcoord));
 	add("texcoord3", static_cast<void (Loader::*)(float, float, float)>(&Loader::texcoord));
 	add("texcoord4", static_cast<void (Loader::*)(float, float, float, float)>(&Loader::texcoord));
+	add("multitexcoord1", static_cast<void (Loader::*)(unsigned, float)>(&Loader::multitexcoord));
+	add("multitexcoord2", static_cast<void (Loader::*)(unsigned, float, float)>(&Loader::multitexcoord));
+	add("multitexcoord3", static_cast<void (Loader::*)(unsigned, float, float, float)>(&Loader::multitexcoord));
+	add("multitexcoord4", static_cast<void (Loader::*)(unsigned, float, float, float, float)>(&Loader::multitexcoord));
 	add("color3",    static_cast<void (Loader::*)(float, float, float)>(&Loader::color));
 	add("color4",    static_cast<void (Loader::*)(float, float, float, float)>(&Loader::color));
 	add("attrib1",   static_cast<void (Loader::*)(unsigned, float)>(&Loader::attrib));
 	add("attrib2",   static_cast<void (Loader::*)(unsigned, float, float)>(&Loader::attrib));
 	add("attrib3",   static_cast<void (Loader::*)(unsigned, float, float, float)>(&Loader::attrib));
 	add("attrib4",   static_cast<void (Loader::*)(unsigned, float, float, float, float)>(&Loader::attrib));
-}
-
-
-void array_element(int i)
-{
-	glArrayElement(i);
-}
-
-void draw_arrays(PrimitiveType mode, int first, unsigned count)
-{
-	glDrawArrays(mode, first, count);
-}
-
-void draw_elements(PrimitiveType mode, unsigned count, DataType type, const void *indices)
-{
-	glDrawElements(mode, count, type, indices);
-}
-
-void draw_range_elements(PrimitiveType mode, unsigned low, unsigned high, unsigned count, DataType type, const void *indices)
-{
-	static RequireVersion _ver(1, 2);
-	glDrawRangeElements(mode, low, high, count, type, indices);
+	add("tangent3",  static_cast<void (Loader::*)(float, float, float)>(&Loader::tangent));
+	add("binormal3", static_cast<void (Loader::*)(float, float, float)>(&Loader::binormal));
 }
 
 } // namespace GL