From 17452f175e26325bea9eeb28f4939c02961df49a Mon Sep 17 00:00:00 2001
From: fgenesis <false.genesis@googlemail.com>
Date: Fri, 25 Aug 2023 00:35:27 +0200
Subject: [PATCH] tiny fixes to VBO

---
 BBGE/VertexBuffer.cpp | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/BBGE/VertexBuffer.cpp b/BBGE/VertexBuffer.cpp
index 5294f57..764b07e 100644
--- a/BBGE/VertexBuffer.cpp
+++ b/BBGE/VertexBuffer.cpp
@@ -77,6 +77,7 @@ void* DynamicGPUBuffer::_ensureBytes(size_t bytes)
 
 void* DynamicGPUBuffer::beginWrite(BufDataType type, size_t newsize, unsigned access)
 {
+    assert(!_d_map);
     _size = newsize;
     _datatype = type;
 
@@ -115,8 +116,10 @@ bool DynamicGPUBuffer::_commitWrite(size_t used)
         if(_d_map)
         {
             assert(used <= _d_cap);
-            _d_map = NULL;
-            return glUnmapBufferARB(_binding); // can fail
+            bool ok = glUnmapBufferARB(_binding); // can fail
+            if(ok)
+                _d_map = NULL;
+            return ok;
         }
         // otherwise, the prev. call to glMapBufferARB failed (or GPUACCESS_HOSTCOPY was set).
         // -> didn't map, but wrote to host memory. upload it.
@@ -255,18 +258,18 @@ void DynamicGPUBuffer::dropBuffer()
     _size = 0;
 }
 
-static unsigned getBoundBuffer(unsigned target)
+/*static unsigned getBoundBuffer(unsigned target)
 {
     int id = 0;
     glGetIntegerv(target, &id);
     return id;
-}
+}*/
 
 void DynamicGPUBuffer::drawElements(unsigned glmode, size_t n, size_t first) const
 {
     assert(_binding == GL_ELEMENT_ARRAY_BUFFER_ARB);
     assert(s_gltype[_datatype & 0xf] == GL_SHORT);
-    assert(getBoundBuffer(GL_ARRAY_BUFFER_BINDING)); // FIXME: this assert is wrong if indices are on the host
+    //assert(getBoundBuffer(GL_ARRAY_BUFFER_BINDING)); // FIXME: this assert is wrong if indices are on the host
 
     unsigned id = _bufid;