X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fmesh.cpp;h=150822ce85e1771a3826332cb19c12922c15a044;hb=6c525eff3eaaa617648a869449ed3bddf9c1936d;hp=316e5a3aa9f72ecbcfb3d09e5c9bc09e3b79dfe0;hpb=47af19a447e6c13257ca2d0aef606ea45f8a0d98;p=libs%2Fgl.git

diff --git a/source/mesh.cpp b/source/mesh.cpp
index 316e5a3a..150822ce 100644
--- a/source/mesh.cpp
+++ b/source/mesh.cpp
@@ -16,7 +16,7 @@ Mesh::Mesh():
 	ibuf(0),
 	vao_id(0),
 	defer_buffers(true),
-	dirty(0),
+	dirty(true),
 	winding(0)
 { }
 
@@ -26,7 +26,7 @@ Mesh::Mesh(const VertexFormat &f):
 	ibuf(0),
 	vao_id(0),
 	defer_buffers(true),
-	dirty(2),
+	dirty(true),
 	winding(0)
 { }
 
@@ -74,49 +74,29 @@ void Mesh::create_buffers()
 		glGenVertexArrays(1, &vao_id);
 }
 
-void Mesh::refresh() const
+void Mesh::setup_vao() const
 {
-	vertices.refresh();
-	if(dirty)
+	Bind bind_vbuf(vbuf, ARRAY_BUFFER);
+
+	const VertexFormat &fmt = vertices.get_format();
+	unsigned stride = get_stride(fmt)*sizeof(float);
+	float *ptr = 0;
+	for(const unsigned char *c=fmt.begin(); c!=fmt.end(); ++c)
 	{
-		if(dirty&1)
-		{
-			for(list<Batch>::const_iterator i=batches.begin(); i!=batches.end(); ++i)
-				i->refresh();
-		}
-
-		if(dirty&2)
-		{
-			Bind bind_vbuf(vbuf, ARRAY_BUFFER);
-
-			bool bind_here = !current();
-			if(bind_here)
-				glBindVertexArray(vao_id);
-
-			const VertexFormat &fmt = vertices.get_format();
-			unsigned stride = get_stride(fmt)*sizeof(float);
-			float *ptr = 0;
-			for(const unsigned char *c=fmt.begin(); c!=fmt.end(); ++c)
-			{
-				unsigned t = get_component_type(*c);
-				if(t>=get_component_type(ATTRIB1))
-					t -= get_component_type(ATTRIB1);
-				unsigned sz = get_component_size(*c);
-				if(*c==COLOR4_UBYTE)
-					glVertexAttribPointer(t, 4, GL_UNSIGNED_BYTE, true, stride, ptr);
-				else
-					glVertexAttribPointer(t, sz, GL_FLOAT, false, stride, ptr);
-				glEnableVertexAttribArray(t);
-				ptr += sz;
-			}
-			glBindBuffer(ELEMENT_ARRAY_BUFFER, ibuf->get_id());
-
-			if(bind_here)
-				glBindVertexArray(0);
-		}
-
-		dirty = 0;
+		unsigned t = get_component_type(*c);
+		if(t>=get_component_type(ATTRIB1))
+			t -= get_component_type(ATTRIB1);
+		unsigned sz = get_component_size(*c);
+		if(*c==COLOR4_UBYTE)
+			glVertexAttribPointer(t, 4, GL_UNSIGNED_BYTE, true, stride, ptr);
+		else
+			glVertexAttribPointer(t, sz, GL_FLOAT, false, stride, ptr);
+		glEnableVertexAttribArray(t);
+		ptr += sz;
 	}
+	glBindBuffer(ELEMENT_ARRAY_BUFFER, ibuf->get_id());
+
+	dirty = false;
 }
 
 unsigned Mesh::get_n_vertices() const
@@ -134,7 +114,6 @@ void Mesh::add_batch(const Batch &b)
 	if(defer_buffers)
 		create_buffers();
 
-	dirty |= 1;
 	if(!batches.empty() && batches.back().can_append(b.get_type()))
 		batches.back().append(b);
 	else
@@ -157,8 +136,6 @@ void Mesh::draw() const
 	if(cur && cur!=this)
 		throw invalid_operation("Mesh::draw");
 
-	refresh();
-
 	if(!current())
 		vertices.apply();
 	Bind bind_ibuf(ibuf, ELEMENT_ARRAY_BUFFER);
@@ -170,8 +147,6 @@ void Mesh::draw() const
 
 void Mesh::draw(Renderer &renderer) const
 {
-	refresh();
-
 	renderer.set_mesh(this);
 	renderer.set_element_buffer(ibuf);
 	renderer.set_winding_test(winding);
@@ -185,9 +160,17 @@ void Mesh::bind() const
 	/* If VAOs are not supported, vao_id is zero and set_current won't get
 	called.  Thus unbind won't try to call a null function either. */
 	if(!vao_id)
+	{
 		unbind();
+		vertices.apply();
+	}
 	else if(set_current(this))
+	{
 		glBindVertexArray(vao_id);
+		vertices.refresh();
+		if(dirty)
+			setup_vao();
+	}
 }
 
 void Mesh::unbind()
@@ -214,7 +197,7 @@ void Mesh::Loader::vertices(const vector<VertexComponent> &c)
 	for(vector<VertexComponent>::const_iterator i=c.begin(); i!=c.end(); ++i)
 		fmt = (fmt, *i);
 	obj.vertices.reset(fmt);
-	obj.dirty |= 2;
+	obj.dirty = true;
 	load_sub(obj.vertices);
 }