X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fvertexarray.cpp;h=9947ac1108a59ef5b5a9a72d9caaa7987ba11782;hb=a40fc85277dba5c34402a0e703d038efd30cc57b;hp=3fca527d75aee2dba990a176a2415f514a537f92;hpb=99b3477d16295c9b90daa0a397bfc375dd3aac4d;p=libs%2Fgl.git diff --git a/source/vertexarray.cpp b/source/vertexarray.cpp index 3fca527d..9947ac11 100644 --- 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,31 @@ void VertexArray::reserve(unsigned n) float *VertexArray::append() { data.insert(data.end(), stride, 0.0f); - set_dirty(); + update_offset(); + 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 { 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"); + + static Require _req(ARB_vertex_shader); const VertexArray *old = current(); /* If the array has been modified, apply it even if it was the last one to @@ -122,27 +119,18 @@ 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; - } - } + const 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(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 *arrays, const vector *old_arrays, const float *base, unsigned stride_bytes) { - unsigned active_tex = 0; unsigned n_arrays = arrays ? arrays->size() : 0; if(old_arrays) n_arrays = max(n_arrays, old_arrays->size()); @@ -156,69 +144,30 @@ void VertexArray::apply_arrays(const vector *arrays, const vector unsigned char comp = (arr ? arr->component : old_arr->component); unsigned sz = get_component_size(comp); unsigned t = get_component_type(comp); - GLenum array_type = 0; - if(t==get_component_type(VERTEX3)) - { - if(arr) - glVertexPointer(sz, GL_FLOAT, stride_bytes, base+arr->offset); - array_type = GL_VERTEX_ARRAY; - } - else if(t==get_component_type(NORMAL3)) - { - if(arr) - glNormalPointer(GL_FLOAT, stride_bytes, base+arr->offset); - array_type = GL_NORMAL_ARRAY; - } - else if(t==get_component_type(COLOR4_FLOAT)) - { - if(arr) - { - if(sz==1) - glColorPointer(4, GL_UNSIGNED_BYTE, stride_bytes, base+arr->offset); - else - glColorPointer(sz, GL_FLOAT, stride_bytes, base+arr->offset); - } - array_type = GL_COLOR_ARRAY; - } - else if(comp>=TEXCOORD1 && comp<=TEXCOORD4+12) - { - t -= get_component_type(TEXCOORD1); - if(t>0 || active_tex) - { - glClientActiveTexture(GL_TEXTURE0+t); - active_tex = t; - } - if(arr) - glTexCoordPointer(sz, GL_FLOAT, stride_bytes, base+arr->offset); - array_type = GL_TEXTURE_COORD_ARRAY; - } - else + + if(t>=get_component_type(ATTRIB1)) + t -= get_component_type(ATTRIB1); + if(arr) { - if(t>=get_component_type(ATTRIB1)) - t -= get_component_type(ATTRIB1); - if(arr) + if(arr->component==COLOR4_UBYTE) + glVertexAttribPointer(t, 4, GL_UNSIGNED_BYTE, true, stride_bytes, base+arr->offset); + else glVertexAttribPointer(t, sz, GL_FLOAT, false, stride_bytes, base+arr->offset); } // Only change enable state if needed if(arr && !old_arr) - { - if(array_type) - glEnableClientState(array_type); - else - glEnableVertexAttribArray(t); - } + glEnableVertexAttribArray(t); else if(old_arr && !arr) - { - if(array_type) - glDisableClientState(array_type); - else - glDisableVertexAttribArray(t); - } + glDisableVertexAttribArray(t); } +} - if(active_tex) - glClientActiveTexture(GL_TEXTURE0); +void VertexArray::unbind() +{ + const VertexArray *old = current(); + if(set_current(0)) + apply_arrays(0, &old->arrays, 0, 0); } @@ -231,6 +180,27 @@ VertexArray::Array::Array(): VertexArray::Loader::Loader(VertexArray &a): VertexArrayBuilder(a) { + add("vertex", static_cast(&Loader::vertex)); + add("vertex", static_cast(&Loader::vertex)); + add("vertex", static_cast(&Loader::vertex)); + add("normal", static_cast(&Loader::normal)); + add("texcoord", static_cast(&Loader::texcoord)); + add("texcoord", static_cast(&Loader::texcoord)); + add("texcoord", static_cast(&Loader::texcoord)); + add("texcoord", static_cast(&Loader::texcoord)); + add("multitexcoord", static_cast(&Loader::multitexcoord)); + add("multitexcoord", static_cast(&Loader::multitexcoord)); + add("multitexcoord", static_cast(&Loader::multitexcoord)); + add("multitexcoord", static_cast(&Loader::multitexcoord)); + add("color", static_cast(&Loader::color)); + add("color", static_cast(&Loader::color)); + add("attrib", static_cast(&Loader::attrib)); + add("attrib", static_cast(&Loader::attrib)); + add("attrib", static_cast(&Loader::attrib)); + add("attrib", static_cast(&Loader::attrib)); + add("tangent", static_cast(&Loader::tangent)); + add("binormal", static_cast(&Loader::binormal)); + add("vertex2", static_cast(&Loader::vertex)); add("vertex3", static_cast(&Loader::vertex)); add("vertex4", static_cast(&Loader::vertex));