X-Git-Url: https://git.ucc.asn.au/?p=ipdf%2Fcode.git;a=blobdiff_plain;f=src%2Fgraphicsbuffer.cpp;h=c3d9620d36138631f25896053995540bef50268f;hp=91c3478669ce641dca4f7e7fe2dff4397f4ab64e;hb=a8297c3461718f2d9afc7a2f8ca620d320ac5f97;hpb=4f60a4a972fa76800580f8731aba54a8fe94e1fb diff --git a/src/graphicsbuffer.cpp b/src/graphicsbuffer.cpp index 91c3478..c3d9620 100644 --- a/src/graphicsbuffer.cpp +++ b/src/graphicsbuffer.cpp @@ -130,11 +130,13 @@ bool GraphicsBuffer::RecreateBuffer(const void *data) { glDeleteBuffers(1, &m_buffer_handle); } - glGenBuffers(1, &m_buffer_handle); - glObjectLabel(GL_BUFFER, m_buffer_handle, -1, m_name); - m_buffer_shape_dirty = false; if (m_buffer_size) + { + glGenBuffers(1, &m_buffer_handle); + glObjectLabel(GL_BUFFER, m_buffer_handle, -1, m_name); + m_buffer_shape_dirty = false; Upload(m_buffer_size, data); + } return true; } @@ -156,6 +158,10 @@ void* GraphicsBuffer::Map(bool read, bool write, bool invalidate) if (m_map_pointer) Warn("Tried to map already mapped buffer!"); + // Intel really doesn't seem to like this. + if (!m_buffer_size) + return (m_map_pointer = 0); + if (!read && m_buffer_usage == BufferUsage::BufferUsageStaticDraw) { @@ -181,6 +187,10 @@ void* GraphicsBuffer::MapRange(int offset, int length, bool read, bool write, bo if (m_map_pointer) Warn("Tried to map already mapped buffer!"); + // This sometimes makes Intel corrupt memory? + if (!length) return (m_map_pointer = 0); + + RecreateBuffer(); Bind(); @@ -224,7 +234,7 @@ void GraphicsBuffer::Upload(size_t length, const void* data) if (!RecreateBuffer(data)) { Bind(); - glBufferData(target, length, data, usage); + glBufferData(target, length+1, data, usage); } if (data != NULL) m_invalidated = false; @@ -243,13 +253,23 @@ void GraphicsBuffer::UploadRange(size_t length, intptr_t offset, const void* dat void GraphicsBuffer::Resize(size_t length) { - glPushDebugGroup(GL_DEBUG_SOURCE_APPLICATION, 0, -1, "Resizing buffer."); - if (m_invalidated) + if (!m_buffer_size) { m_buffer_size = length; + return; + } + if (m_invalidated && m_buffer_size >= length) + { + m_buffer_size = length; + } + else if (length <= m_buffer_size) + { + // Don't need to do anything. } else { + glPushDebugGroup(GL_DEBUG_SOURCE_APPLICATION, 0, -1, "Resizing buffer."); + glDebugMessageControl(GL_DONT_CARE, GL_DONT_CARE, GL_DEBUG_SEVERITY_HIGH, 0, NULL, GL_TRUE); // Create a new buffer and copy the old data into it. UnMap(); GLuint old_buffer = m_buffer_handle; @@ -260,8 +280,8 @@ void GraphicsBuffer::Resize(size_t length) glCopyBufferSubData(GL_COPY_READ_BUFFER, GL_COPY_WRITE_BUFFER, 0, 0, m_buffer_size); glDeleteBuffers(1, &old_buffer); m_buffer_size = length; + glPopDebugGroup(); } - glPopDebugGroup(); } void GraphicsBuffer::Bind() const