Remove support for legacy OpenGL features

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

diff --git a/source/batch.cpp b/source/batch.cpp
index 956fe1309e80ad0b65a714b2ac000b5162c05348..b61da0bac79aee1507ac536efa46815c6a5c127a 100644 (file)
--- a/source/batch.cpp
+++ b/source/batch.cpp
@@ -1,9 +1,11 @@
+#include <msp/gl/extensions/arb_draw_instanced.h>
 #include <msp/gl/extensions/ext_draw_range_elements.h>
-#include <msp/gl/extensions/nv_primitive_restart.h>
+#include <msp/gl/extensions/msp_primitive_restart.h>
 #include "batch.h"
 #include "bindable.h"
 #include "buffer.h"
 #include "error.h"
+#include "mesh.h"
 #include "vertexarray.h"
 
 using namespace std;
@@ -57,11 +59,7 @@ Batch::Batch(PrimitiveType t):
        min_index(0),
        max_index(0),
        restart(false)
-{
-       /* XXX Should probably provide a fallback to glDrawElements since this class
-       is pretty much required to render anything. */
-       static Require _req(EXT_draw_range_elements);
-}
+{ }
 
 Batch::~Batch()
 {
@@ -97,17 +95,17 @@ void Batch::set_data_type(DataType t)
 Batch &Batch::append(unsigned i)
 {
        append_index(i);
-       
+
        update_offset();
        dirty = true;
 
        return *this;
 }
 
-void Batch::append(const vector<unsigned> &ind)
+Batch &Batch::append(const vector<unsigned> &ind)
 {
        if(ind.empty())
-               return;
+               return *this;
 
        data.reserve(data.size()+ind.size()*get_index_size());
        for(vector<unsigned>::const_iterator i=ind.begin(); i!=ind.end(); ++i)
@@ -115,31 +113,33 @@ void Batch::append(const vector<unsigned> &ind)
 
        update_offset();
        dirty = true;
+
+       return *this;
 }
 
 bool Batch::can_append(PrimitiveType other_type)
 {
        if(other_type!=prim_type)
                return false;
-       else if(prim_type==LINE_STRIP || prim_type==LINE_LOOP || prim_type==TRIANGLE_FAN || prim_type==POLYGON)
-               return NV_primitive_restart;
+       else if(prim_type==LINE_STRIP || prim_type==LINE_LOOP || prim_type==TRIANGLE_FAN)
+               return MSP_primitive_restart;
        else
                return true;
 }
 
-void Batch::append(const Batch &other)
+Batch &Batch::append(const Batch &other)
 {
        if(other.prim_type!=prim_type)
                throw invalid_argument("Batch::append");
-       if(prim_type==LINE_STRIP || prim_type==LINE_LOOP || prim_type==TRIANGLE_FAN || prim_type==POLYGON)
-               static Require _req(NV_primitive_restart);
+       if(prim_type==LINE_STRIP || prim_type==LINE_LOOP || prim_type==TRIANGLE_FAN)
+               static Require _req(MSP_primitive_restart);
 
        if(other.data.empty())
-               return;
+               return *this;
 
-       if(prim_type==POINTS || prim_type==LINES || prim_type==TRIANGLES || prim_type==QUADS)
+       if(prim_type==POINTS || prim_type==LINES || prim_type==TRIANGLES)
                ;
-       else if(NV_primitive_restart)
+       else if(MSP_primitive_restart)
        {
                restart = true;
                if(data_type==UNSIGNED_SHORT)
@@ -156,13 +156,6 @@ void Batch::append(const Batch &other)
                if(size()&1)
                        append(other.get_index(0));
        }
-       else if(prim_type==QUAD_STRIP)
-       {
-               append(get_index(size()-1));
-               append(get_index(size()-1));
-               append(other.get_index(0));
-               append(other.get_index(0));
-       }
 
        unsigned count = other.size();
        for(unsigned i=0; i<count; ++i)
@@ -170,6 +163,8 @@ void Batch::append(const Batch &other)
 
        update_offset();
        dirty = true;
+
+       return *this;
 }
 
 void Batch::append_index(unsigned i)
@@ -195,11 +190,6 @@ void Batch::append_index(unsigned i)
                data.push_back(i);
 }
 
-void Batch::upload_data() const
-{
-       get_buffer()->sub_data(get_offset(), data.size(), &data[0]);
-}
-
 unsigned Batch::get_index_size() const
 {
        if(data_type==UNSIGNED_SHORT)
@@ -220,6 +210,27 @@ unsigned Batch::get_index(unsigned i) const
 }
 
 void Batch::draw() const
+{
+       BindRestore _bind_ibuf(get_buffer(), ELEMENT_ARRAY_BUFFER);
+       const void *data_ptr = setup_draw();
+
+       if(EXT_draw_range_elements)
+               glDrawRangeElements(prim_type, min_index, max_index, size(), data_type, data_ptr);
+       else
+               glDrawElements(prim_type, size(), data_type, data_ptr);
+}
+
+void Batch::draw_instanced(unsigned count) const
+{
+       static Require req(ARB_draw_instanced);
+
+       BindRestore _bind_ibuf(get_buffer(), ELEMENT_ARRAY_BUFFER);
+       const void *data_ptr = setup_draw();
+
+       glDrawElementsInstanced(prim_type, size(), data_type, data_ptr, count);
+}
+
+const void *Batch::setup_draw() const
 {
        if(restart)
        {
@@ -232,31 +243,34 @@ void Batch::draw() const
                        index = 0xFF;
 
                if(index!=restart_index)
-               {
-                       if(!restart_index)
-                               glEnableClientState(GL_PRIMITIVE_RESTART_NV);
-                       glPrimitiveRestartIndexNV(index);
-                       restart_index = index;
-               }
+                       set_restart_index(index);
        }
        else if(restart_index && restart_index<=max_index)
-       {
-               glDisableClientState(GL_PRIMITIVE_RESTART_NV);
-               restart_index = 0;
-       }
+               set_restart_index(0);
 
        if(get_buffer())
        {
                if(dirty)
                        update_buffer();
 
-               BufferAlias<ELEMENT_ARRAY_BUFFER> alias(*get_buffer());
-               Bind bind_ibuf(alias, true);
+               return reinterpret_cast<const void *>(get_offset());
+       }
+       else
+               return &data[0];
+}
 
-               glDrawRangeElements(prim_type, min_index, max_index, size(), data_type, reinterpret_cast<void *>(get_offset()));
+void Batch::set_restart_index(unsigned index)
+{
+       if(index>0)
+       {
+               if(!restart_index)
+                       glEnable(GL_PRIMITIVE_RESTART);
+               glPrimitiveRestartIndex(index);
        }
        else
-               glDrawRangeElements(prim_type, min_index, max_index, size(), data_type, &data[0]);
+               glDisable(GL_PRIMITIVE_RESTART);
+
+       restart_index = index;
 }