]> git.tdb.fi Git - libs/gl.git/blobdiff - source/batch.cpp
Remove the deprecated ProgramBuilder class
[libs/gl.git] / source / batch.cpp
index 37e4801ec82a6356a595a4dea62de926bf790d3e..f14cc307ff71b92deb212e9f759ac70f1c8d69f0 100644 (file)
@@ -1,7 +1,5 @@
-#include <msp/gl/extensions/ext_draw_range_elements.h>
-#include <msp/gl/extensions/msp_legacy_features.h>
+#include <msp/gl/extensions/arb_draw_instanced.h>
 #include <msp/gl/extensions/msp_primitive_restart.h>
-#include <msp/gl/extensions/nv_primitive_restart.h>
 #include "batch.h"
 #include "bindable.h"
 #include "buffer.h"
@@ -14,7 +12,7 @@ using namespace std;
 namespace {
 
 template<typename T>
-void append(vector<unsigned char> &data, T i)
+void append(vector<Msp::UInt8> &data, T i)
 {
        data.insert(data.end(), sizeof(T), 0);
        *(T *)(&data[data.size()-sizeof(T)]) = i;
@@ -30,7 +28,7 @@ U convert(T n)
 }
 
 template<typename T, typename U>
-void expand(vector<unsigned char> &data)
+void expand(vector<Msp::UInt8> &data)
 {
        unsigned count = data.size()/sizeof(T);
        data.resize(count*sizeof(U));
@@ -39,7 +37,7 @@ void expand(vector<unsigned char> &data)
 }
 
 template<typename T, typename U>
-void shrink(vector<unsigned char> &data)
+void shrink(vector<Msp::UInt8> &data)
 {
        unsigned count = data.size()/sizeof(T);
        for(unsigned i=0; i<count; ++i)
@@ -56,8 +54,7 @@ unsigned Batch::restart_index = 0;
 
 Batch::Batch(PrimitiveType t):
        prim_type(t),
-       data_type(UNSIGNED_BYTE),
-       min_index(0),
+       index_type(UNSIGNED_SHORT),
        max_index(0),
        restart(false)
 { }
@@ -66,29 +63,19 @@ Batch::~Batch()
 {
 }
 
-void Batch::set_data_type(DataType t)
+void Batch::set_index_type(DataType t)
 {
-       if(t!=UNSIGNED_BYTE && t!=UNSIGNED_SHORT && t!=UNSIGNED_INT)
+       if(t!=UNSIGNED_SHORT && t!=UNSIGNED_INT)
                throw invalid_argument("Batch::set_data_type");
-       if(t==UNSIGNED_BYTE && max_index>0xFE)
-               throw invalid_operation("Batch::set_data_type");
-       else if(t==UNSIGNED_SHORT && max_index>0xFFFE)
+       if(t==UNSIGNED_SHORT && max_index>0xFFFE)
                throw invalid_operation("Batch::set_data_type");
 
-       if(data_type==UNSIGNED_BYTE && t==UNSIGNED_SHORT)
-               expand<unsigned char, unsigned short>(data);
-       else if(data_type==UNSIGNED_BYTE && t==UNSIGNED_INT)
-               expand<unsigned char, unsigned>(data);
-       else if(data_type==UNSIGNED_SHORT && t==UNSIGNED_INT)
-               expand<unsigned short, unsigned>(data);
-       else if(data_type==UNSIGNED_INT && t==UNSIGNED_BYTE)
-               shrink<unsigned, unsigned char>(data);
-       else if(data_type==UNSIGNED_INT && t==UNSIGNED_SHORT)
-               shrink<unsigned, unsigned short>(data);
-       else if(data_type==UNSIGNED_SHORT && t==UNSIGNED_BYTE)
-               shrink<unsigned short, unsigned char>(data);
-
-       data_type = t;
+       if(index_type==UNSIGNED_SHORT && t==UNSIGNED_INT)
+               expand<UInt16, UInt32>(data);
+       else if(index_type==UNSIGNED_INT && t==UNSIGNED_SHORT)
+               shrink<UInt32, UInt16>(data);
+
+       index_type = t;
        update_offset();
        dirty = true;
 }
@@ -123,7 +110,7 @@ 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)
-               return MSP_primitive_restart || NV_primitive_restart;
+               return MSP_primitive_restart;
        else
                return true;
 }
@@ -133,29 +120,22 @@ 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)
-       {
-               if(!MSP_primitive_restart)
-               {
-                       static Require _req(NV_primitive_restart);
-                       // Make sure we have glEnable/DisableClientState as well
-                       static Require _req2(MSP_legacy_features);
-               }
-       }
+               static Require _req(MSP_primitive_restart);
 
        if(other.data.empty())
                return *this;
 
-       if(prim_type==POINTS || prim_type==LINES || prim_type==TRIANGLES || prim_type==QUADS)
+       // TODO allow appending triangles to a triangle strip
+
+       if(prim_type==POINTS || prim_type==LINES || prim_type==TRIANGLES)
                ;
-       else if(MSP_primitive_restart || NV_primitive_restart)
+       else if(MSP_primitive_restart)
        {
                restart = true;
-               if(data_type==UNSIGNED_SHORT)
-                       ::append<unsigned short>(data, 0xFFFF);
-               else if(data_type==UNSIGNED_INT)
-                       ::append<unsigned>(data, 0xFFFFFFFF);
+               if(index_type==UNSIGNED_INT)
+                       ::append<UInt32>(data, 0xFFFFFFFF);
                else
-                       data.push_back(0xFF);
+                       ::append<UInt16>(data, 0xFFFF);
        }
        else if(prim_type==TRIANGLE_STRIP)
        {
@@ -164,13 +144,6 @@ Batch &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)
@@ -185,56 +158,58 @@ Batch &Batch::append(const Batch &other)
 void Batch::append_index(unsigned i)
 {
        if(data.empty())
-               min_index = max_index = i;
+               max_index = i;
        else
-       {
-               min_index = min(min_index, i);
                max_index = max(max_index, i);
-       }
 
-       if((data_type==UNSIGNED_BYTE || data_type==UNSIGNED_SHORT) && max_index>0xFFFE)
-               set_data_type(UNSIGNED_INT);
-       else if(data_type==UNSIGNED_BYTE && max_index>0xFE)
-               set_data_type(UNSIGNED_SHORT);
+       if(index_type==UNSIGNED_SHORT && max_index>0xFFFE)
+               set_index_type(UNSIGNED_INT);
 
-       if(data_type==UNSIGNED_SHORT)
-               ::append<unsigned short>(data, i);
-       else if(data_type==UNSIGNED_INT)
-               ::append<unsigned>(data, i);
+       if(index_type==UNSIGNED_INT)
+               ::append<UInt32>(data, i);
        else
-               data.push_back(i);
+               ::append<UInt16>(data, i);
 }
 
 unsigned Batch::get_index_size() const
 {
-       if(data_type==UNSIGNED_SHORT)
-               return sizeof(unsigned short);
-       else if(data_type==UNSIGNED_INT)
-               return sizeof(unsigned);
-       return sizeof(unsigned char);
+       return (index_type==UNSIGNED_INT ? sizeof(UInt32) : sizeof(UInt16));
 }
 
 unsigned Batch::get_index(unsigned i) const
 {
-       if(data_type==UNSIGNED_SHORT)
-               return *(unsigned short *)&data[i*sizeof(unsigned short)];
-       else if(data_type==UNSIGNED_INT)
-               return *(unsigned *)&data[i*sizeof(unsigned )];
+       if(index_type==UNSIGNED_INT)
+               return *(UInt32 *)&data[i*sizeof(UInt32)];
        else
-               return data[i];
+               return *(UInt16 *)&data[i*sizeof(UInt16)];
 }
 
 void Batch::draw() const
 {
+       BindRestore _bind_ibuf(get_buffer(), ELEMENT_ARRAY_BUFFER);
+       const void *data_ptr = setup_draw();
+
+       glDrawElements(prim_type, size(), index_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(), index_type, data_ptr, count);
+}
+
+const void *Batch::setup_draw() const
+{
+       if(!get_buffer())
+               throw invalid_operation("Batch::setup_draw");
+
        if(restart)
        {
-               unsigned index;
-               if(data_type==UNSIGNED_SHORT)
-                       index = 0xFFFF;
-               else if(data_type==UNSIGNED_INT)
-                       index = 0xFFFFFFFF;
-               else
-                       index = 0xFF;
+               unsigned index = (index_type==UNSIGNED_INT ? 0xFFFFFFFF : 0xFFFF);
 
                if(index!=restart_index)
                        set_restart_index(index);
@@ -242,49 +217,21 @@ void Batch::draw() const
        else if(restart_index && restart_index<=max_index)
                set_restart_index(0);
 
-       const Buffer *ibuf = get_buffer();
-       const void *data_ptr;
-       BindRestore _bind_ibuf(ibuf, ELEMENT_ARRAY_BUFFER);
-       if(ibuf)
-       {
-               if(dirty)
-                       update_buffer();
-
-               data_ptr = reinterpret_cast<const void *>(get_offset());
-       }
-       else
-               data_ptr = &data[0];
+       refresh();
 
-       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);
+       return reinterpret_cast<const void *>(get_offset());
 }
 
 void Batch::set_restart_index(unsigned index)
 {
-       if(MSP_primitive_restart)
+       if(index>0)
        {
-               if(index>0)
-               {
-                       if(!restart_index)
-                               glEnable(GL_PRIMITIVE_RESTART);
-                       glPrimitiveRestartIndex(index);
-               }
-               else
-                       glDisable(GL_PRIMITIVE_RESTART);
+               if(!restart_index)
+                       glEnable(GL_PRIMITIVE_RESTART);
+               glPrimitiveRestartIndex(index);
        }
        else
-       {
-               if(index>0)
-               {
-                       if(!restart_index)
-                               glEnableClientState(GL_PRIMITIVE_RESTART_NV);
-                       glPrimitiveRestartIndexNV(index);
-               }
-               else
-                       glDisableClientState(GL_PRIMITIVE_RESTART_NV);
-       }
+               glDisable(GL_PRIMITIVE_RESTART);
 
        restart_index = index;
 }