Rework ProgramData to do less unnecessary work

[libs/gl.git] / source / vertexarray.cpp

diff --git a/source/vertexarray.cpp b/source/vertexarray.cpp
index 3fca527d75aee2dba990a176a2415f514a537f92..042f58c758b41ac48af3fccd52dcb7ec5f75440b 100644 (file)
--- a/source/vertexarray.cpp
+++ b/source/vertexarray.cpp
@@ -3,6 +3,7 @@
 #include "buffer.h"
 #include "error.h"
 #include "gl.h"
+#include "mesh.h"
 #include "vertexarray.h"
 
 using namespace std;
@@ -10,21 +11,17 @@ using namespace std;
 namespace Msp {
 namespace GL {
 
-VertexArray::VertexArray(const VertexFormat &f):
-       dirty(false)
+VertexArray::VertexArray(const VertexFormat &f)
 {
        reset(f);
 }
 
 VertexArray::~VertexArray()
 {
+       /* Unbind accesses the current VertexArray, so a call from ~Bindable would
+       try to access destroyed data. */
        if(current()==this)
-       {
-               /* We must deactivate arrays here, or apply() would try to access deleted
-               data on the next invocation. */
-               set_current(0);
-               apply_arrays(0, &arrays, 0, 0);
-       }
+               unbind();
 }
 
 void VertexArray::reset(const VertexFormat &f)
@@ -35,7 +32,7 @@ void VertexArray::reset(const VertexFormat &f)
 
        arrays.clear();
 
-       unsigned offset = 0;
+       unsigned offs = 0;
        for(const unsigned char *c=format.begin(); c!=format.end(); ++c)
        {
                unsigned slot = get_array_slot(*c);
@@ -44,9 +41,9 @@ void VertexArray::reset(const VertexFormat &f)
 
                Array &arr = arrays[slot];
                arr.component = *c;
-               arr.offset = offset;
+               arr.offset = offs;
 
-               offset += get_component_size(*c);
+               offs += get_component_size(*c);
        }
 }
 
@@ -75,12 +72,6 @@ unsigned VertexArray::get_array_slot(unsigned char comp)
        }
 }
 
-void VertexArray::use_vertex_buffer(Buffer *b)
-{
-       vbuf = b;
-       dirty = true;
-}
-
 void VertexArray::clear()
 {
        data.clear();
@@ -94,25 +85,28 @@ void VertexArray::reserve(unsigned n)
 float *VertexArray::append()
 {
        data.insert(data.end(), stride, 0.0f);
-       set_dirty();
+       dirty = true;
        return &*(data.end()-stride);
 }
 
 float *VertexArray::modify(unsigned i)
 {
-       set_dirty();
+       dirty = true;
        return &data[0]+i*stride;
 }
 
-void VertexArray::set_dirty()
+unsigned VertexArray::get_data_size() const
 {
-       dirty = true;
+       return data.size()*sizeof(float);
 }
 
-void VertexArray::apply() const
+void VertexArray::bind() const
 {
        if(format.empty())
                throw invalid_operation("VertexArray::apply");
+       // Don't mess up the vertex array object of a mesh
+       if(Mesh::current())
+               throw invalid_operation("VertexArray::apply");
 
        const VertexArray *old = current();
        /* If the array has been modified, apply it even if it was the last one to
@@ -122,22 +116,14 @@ void VertexArray::apply() const
        if(!set_current(this) && !dirty)
                return;
 
-       if(vbuf)
-       {
-               vbuf->bind_to(ARRAY_BUFFER);
-               if(dirty)
-               {
-                       vbuf->data(data.size()*sizeof(float), &data[0]);
-                       dirty = false;
-               }
-       }
+       Buffer *vbuf = get_buffer();
+       Bind _bind_vbuf(vbuf, ARRAY_BUFFER);
+       if(vbuf && dirty)
+               update_buffer();
 
-       const float *base = (vbuf ? 0 : &data[0]);
+       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);
-
-       if(vbuf)
-               Buffer::unbind_from(ARRAY_BUFFER);
 }
 
 void VertexArray::apply_arrays(const vector<Array> *arrays, const vector<Array> *old_arrays, const float *base, unsigned stride_bytes)
@@ -221,6 +207,13 @@ void VertexArray::apply_arrays(const vector<Array> *arrays, const vector<Array>
                glClientActiveTexture(GL_TEXTURE0);
 }
 
+void VertexArray::unbind()
+{
+       const VertexArray *old = current();
+       if(set_current(0))
+               apply_arrays(0, &old->arrays, 0, 0);
+}
+
 
 VertexArray::Array::Array():
        component(0),