]> git.tdb.fi Git - libs/gl.git/blobdiff - source/mesh.cpp
Some tweaks to Mesh::add_batch
[libs/gl.git] / source / mesh.cpp
index 301c00f94b331fc6c87087bc9f79a8b558c675a0..8f7911219ec68afa3cb9048d35960bef3462437c 100644 (file)
@@ -1,11 +1,12 @@
 #include <msp/gl/extensions/arb_vertex_array_object.h>
+#include <msp/gl/extensions/arb_vertex_buffer_object.h>
 #include <msp/gl/extensions/arb_vertex_shader.h>
-#include <msp/gl/extensions/nv_primitive_restart.h>
 #include "buffer.h"
 #include "error.h"
 #include "mesh.h"
 #include "renderer.h"
 #include "resourcemanager.h"
+#include "vertexsetup.h"
 
 using namespace std;
 
@@ -28,9 +29,6 @@ void Mesh::init(ResourceManager *rm)
 {
        vbuf = 0;
        ibuf = 0;
-       vao_id = 0;
-       defer_buffers = true;
-       dirty = true;
        disallow_rendering = false;
        winding = 0;
 
@@ -40,10 +38,9 @@ void Mesh::init(ResourceManager *rm)
 
 Mesh::~Mesh()
 {
+       set_manager(0);
        delete vbuf;
        delete ibuf;
-       if(vao_id)
-               glDeleteVertexArrays(1, &vao_id);
 }
 
 void Mesh::clear()
@@ -52,24 +49,10 @@ void Mesh::clear()
        batches.clear();
 }
 
-void Mesh::use_buffers(bool b)
-{
-       defer_buffers = false;
-       if(b)
-               create_buffers();
-       else
-       {
-               vertices.use_buffer(0);
-               delete vbuf;
-               vbuf = 0;
-               delete ibuf;
-               ibuf = 0;
-       }
-}
-
 void Mesh::create_buffers()
 {
-       defer_buffers = false;
+       if(vbuf && ibuf)
+               return;
 
        if(!vbuf)
                vbuf = new Buffer(ARRAY_BUFFER);
@@ -78,33 +61,8 @@ void Mesh::create_buffers()
        if(!ibuf)
                ibuf = new Buffer(ELEMENT_ARRAY_BUFFER);
 
-       if(ARB_vertex_array_object && !vao_id)
-               glGenVertexArrays(1, &vao_id);
-}
-
-void Mesh::setup_vao() const
-{
-       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)
-       {
-               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;
+       vtx_setup.set_vertex_array(vertices);
+       vtx_setup.set_index_buffer(*ibuf);
 }
 
 unsigned Mesh::get_n_vertices() const
@@ -119,16 +77,36 @@ float *Mesh::modify_vertex(unsigned i)
 
 void Mesh::add_batch(const Batch &b)
 {
-       if(defer_buffers)
-               create_buffers();
+       create_buffers();
 
-       if(!batches.empty() && batches.back().can_append(b.get_type()))
+       if(batches.empty())
+       {
+               batches.push_back(b);
+               batches.back().use_buffer(ibuf);
+       }
+       else if(batches.back().can_append(b.get_type()))
                batches.back().append(b);
        else
        {
-               Batch *prev = (batches.empty() ? 0 : &batches.back());
+               bool reallocate = (batches.size()==batches.capacity());
+               if(reallocate)
+               {
+                       for(vector<Batch>::iterator i=batches.end(); i!=batches.begin(); )
+                               (--i)->use_buffer(0);
+               }
+
+               Batch *prev = &batches.back();
                batches.push_back(b);
-               if(ibuf)
+               if(reallocate)
+               {
+                       prev = 0;
+                       for(vector<Batch>::iterator i=batches.begin(); i!=batches.end(); ++i)
+                       {
+                               i->use_buffer(ibuf, prev);
+                               prev = &*i;
+                       }
+               }
+               else
                        batches.back().use_buffer(ibuf, prev);
        }
 }
@@ -138,29 +116,20 @@ void Mesh::set_winding(const WindingTest *w)
        winding = w;
 }
 
-void Mesh::draw() const
+void Mesh::draw(Renderer &renderer) const
 {
-       const Mesh *cur = current();
-       if(cur && cur!=this)
-               throw invalid_operation("Mesh::draw");
-
-       if(manager)
-       {
-               manager->resource_used(*this);
-               if(disallow_rendering)
-                       return;
-       }
+       draw(renderer, 0, 0);
+}
 
-       if(!current())
-               vertices.apply();
-       Bind bind_ibuf(ibuf, ELEMENT_ARRAY_BUFFER);
-       Bind bind_winding(winding);
+void Mesh::draw_instanced(Renderer &renderer, const VertexSetup &vs, unsigned count) const
+{
+       if(vs.get_vertex_array()!=&vertices)
+               throw invalid_argument("Mesh::draw_instanced");
 
-       for(list<Batch>::const_iterator i=batches.begin(); i!=batches.end(); ++i)
-               i->draw();
+       draw(renderer, &vs, count);
 }
 
-void Mesh::draw(Renderer &renderer) const
+void Mesh::draw(Renderer &renderer, const VertexSetup *vs, unsigned count) const
 {
        if(manager)
        {
@@ -169,38 +138,22 @@ void Mesh::draw(Renderer &renderer) const
                        return;
        }
 
-       renderer.set_mesh(this);
+       renderer.set_vertex_setup(vs ? vs : &vtx_setup);
        renderer.set_winding_test(winding);
 
-       for(list<Batch>::const_iterator i=batches.begin(); i!=batches.end(); ++i)
-               renderer.draw(*i);
-}
-
-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)
+       if(!count)
        {
-               unbind();
-               vertices.apply(false);
+               for(vector<Batch>::const_iterator i=batches.begin(); i!=batches.end(); ++i)
+                       renderer.draw(*i);
        }
-       else if(set_current(this))
+       else
        {
-               glBindVertexArray(vao_id);
-               vertices.refresh();
-               if(dirty)
-                       setup_vao();
+               for(vector<Batch>::const_iterator i=batches.begin(); i!=batches.end(); ++i)
+                       renderer.draw_instanced(*i, count);
        }
 }
 
-void Mesh::unbind()
-{
-       if(set_current(0))
-               glBindVertexArray(0);
-}
-
-Resource::AsyncLoader *Mesh::load(IO::Seekable &io)
+Resource::AsyncLoader *Mesh::load(IO::Seekable &io, const Resources *)
 {
        return new AsyncLoader(*this, io);
 }
@@ -222,7 +175,6 @@ void Mesh::unload()
        batches.clear();
        delete vbuf;
        delete ibuf;
-       defer_buffers = (vbuf || ibuf);
        vbuf = 0;
        ibuf = 0;
 }
@@ -245,7 +197,9 @@ 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 = true;
+       if(obj.vbuf)
+               // Set it again to force the vertex setup to update
+               obj.vtx_setup.set_vertex_array(obj.vertices);
        load_sub(obj.vertices);
 }
 
@@ -272,12 +226,8 @@ Mesh::AsyncLoader::AsyncLoader(Mesh &m, IO::Seekable &i):
        index_updater(0),
        phase(0)
 {
-       // Make sure the extension is initialized in the rendering thread.
-       (bool)NV_primitive_restart;
-
        mesh.disallow_rendering = true;
-       if(mesh.defer_buffers)
-               mesh.create_buffers();
+       mesh.create_buffers();
 }
 
 Mesh::AsyncLoader::~AsyncLoader()