X-Git-Url: https://git.ucc.asn.au/?p=ipdf%2Fcode.git;a=blobdiff_plain;f=src%2Fgraphicsbuffer.cpp;h=217fa9a00129e859eb9b5bc16d6d02ea8a686ab9;hp=37d9b12f82ceae844098fbf710d01e110bf00cb0;hb=888817a67a9d840be66b52811b01eb77f10ff3e6;hpb=af1f07330cb8f4d448cc75ee720b633338eba1e8 diff --git a/src/graphicsbuffer.cpp b/src/graphicsbuffer.cpp index 37d9b12..217fa9a 100644 --- a/src/graphicsbuffer.cpp +++ b/src/graphicsbuffer.cpp @@ -36,7 +36,7 @@ static GLenum BufferUsageToGLUsage(GraphicsBuffer::BufferUsage buffer_usage) usage = GL_STREAM_COPY; break; default: - SDL_assert(false && "Unknown buffer usage type."); + SDL_assert(false && "Unknown buffer usage type."); //WTF? usage = GL_DYNAMIC_DRAW; } return usage; @@ -56,6 +56,8 @@ static GLenum BufferTypeToGLType(GraphicsBuffer::BufferType buffer_type) return GL_PIXEL_UNPACK_BUFFER; case GraphicsBuffer::BufferTypeUniform: return GL_UNIFORM_BUFFER; + case GraphicsBuffer::BufferTypeTexture: + return GL_TEXTURE_BUFFER; case GraphicsBuffer::BufferTypeDrawIndirect: return GL_DRAW_INDIRECT_BUFFER; default: @@ -66,11 +68,13 @@ static GLenum BufferTypeToGLType(GraphicsBuffer::BufferType buffer_type) GraphicsBuffer::GraphicsBuffer() { m_invalidated = true; - m_map_pointer = nullptr; + m_map_pointer = NULL; m_buffer_size = 0; m_buffer_shape_dirty = true; m_buffer_handle = 0; m_buffer_usage = BufferUsageDynamicDraw; + m_faking_map = false; + m_name = "Unnamed Buffer"; SetUsage(BufferUsageStaticDraw); } @@ -88,6 +92,11 @@ void GraphicsBuffer::SetType(GraphicsBuffer::BufferType bufType) m_buffer_type = bufType; } +void GraphicsBuffer::SetName(const char *name) +{ + m_name = name; +} + void GraphicsBuffer::SetUsage(GraphicsBuffer::BufferUsage bufUsage) { if (bufUsage != m_buffer_usage) @@ -103,28 +112,30 @@ void GraphicsBuffer::Invalidate() if (!m_buffer_shape_dirty) { // Orphan the block of memory we're pointing to. - Upload(m_buffer_size, nullptr); + Upload(m_buffer_size, NULL); } // Apparently not supported. //glInvalidateBufferData(m_buffer_handle); } -void GraphicsBuffer::RecreateBuffer() +bool GraphicsBuffer::RecreateBuffer(const void *data) { // If the buffer is not dirty, don't recreate it. - if (!m_buffer_shape_dirty) return; + if (!m_buffer_shape_dirty) return false; // If the buffer is mapped, don't recreate it. - if (m_map_pointer) return; + if (!m_faking_map && m_map_pointer) return false; // If the buffer has data in it we need, don't recreate it. - if (!m_invalidated) return; + if (!m_invalidated) return false; if (m_buffer_handle) { 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) - Upload(m_buffer_size, nullptr); + Upload(m_buffer_size, data); + return true; } void* GraphicsBuffer::Map(bool read, bool write, bool invalidate) @@ -145,6 +156,14 @@ void* GraphicsBuffer::Map(bool read, bool write, bool invalidate) if (m_map_pointer) Warn("Tried to map already mapped buffer!"); + + if (!read && m_buffer_usage == BufferUsage::BufferUsageStaticDraw) + { + m_map_pointer = malloc(m_buffer_size); + m_faking_map = true; + return m_map_pointer; + } + RecreateBuffer(); Bind(); @@ -174,9 +193,23 @@ void GraphicsBuffer::UnMap() { GLenum target = BufferTypeToGLType(m_buffer_type); + // If we're not mapped, unmapping is a no-op. + if (!m_map_pointer) + return; + + if (m_faking_map) + { + Upload(m_buffer_size, m_map_pointer); + free(m_map_pointer); + m_map_pointer = NULL; + m_invalidated = false; + m_faking_map = false; + return; + } + Bind(); glUnmapBuffer(target); - m_map_pointer = nullptr; + m_map_pointer = NULL; m_invalidated = false; } @@ -187,13 +220,14 @@ void GraphicsBuffer::Upload(size_t length, const void* data) GLenum usage = BufferUsageToGLUsage(m_buffer_usage); m_invalidated = true; - RecreateBuffer(); - - Bind(); - glBufferData(target, length, data, usage); - if (data != nullptr) - m_invalidated = false; m_buffer_size = length; + if (!RecreateBuffer(data)) + { + Bind(); + glBufferData(target, length, data, usage); + } + if (data != NULL) + m_invalidated = false; } void GraphicsBuffer::UploadRange(size_t length, intptr_t offset, const void* data) @@ -212,20 +246,22 @@ void GraphicsBuffer::Resize(size_t length) if (m_invalidated) { m_buffer_size = length; - RecreateBuffer(); } 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; glGenBuffers(1, &m_buffer_handle); - Upload(length, nullptr); + Upload(length, NULL); glBindBuffer(GL_COPY_READ_BUFFER, old_buffer); glBindBuffer(GL_COPY_WRITE_BUFFER, m_buffer_handle); glCopyBufferSubData(GL_COPY_READ_BUFFER, GL_COPY_WRITE_BUFFER, 0, 0, m_buffer_size); glDeleteBuffers(1, &old_buffer); m_buffer_size = length; + glPopDebugGroup(); } } @@ -237,3 +273,7 @@ void GraphicsBuffer::Bind() const glBindBuffer(BufferTypeToGLType(m_buffer_type), m_buffer_handle); } +void GraphicsBuffer::BindRange(size_t start, size_t size) const +{ + glBindBufferRange(BufferTypeToGLType(m_buffer_type), 0, m_buffer_handle, start, size); +}