aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRoman Kiryanov <rkir@google.com>2018-09-19 13:38:53 -0700
committerLingfeng Yang <lfy@google.com>2018-09-21 11:46:28 -0700
commitfb903d68a2e61dbb85ab820a1fb1df36a11285ec (patch)
treed4225aee9021ed18cd36b18a2070271cd700d08d
parentd2cb44dc2a65f35b786dc77160eb3dc16fcff3d6 (diff)
downloaddevice_generic_goldfish-opengl-fb903d68a2e61dbb85ab820a1fb1df36a11285ec.tar.gz
device_generic_goldfish-opengl-fb903d68a2e61dbb85ab820a1fb1df36a11285ec.tar.bz2
device_generic_goldfish-opengl-fb903d68a2e61dbb85ab820a1fb1df36a11285ec.zip
Fix const correctness around BufferData
Bug: 116046430 Test: make -j Change-Id: Ie026fc0bda83ab6e5251f5598a5613cc0d4068a0 Signed-off-by: Roman Kiryanov <rkir@google.com>
-rwxr-xr-xshared/OpenglCodecCommon/GLSharedGroup.cpp8
-rwxr-xr-xshared/OpenglCodecCommon/GLSharedGroup.h8
-rwxr-xr-xsystem/GLESv2_enc/GL2Encoder.cpp4
3 files changed, 10 insertions, 10 deletions
diff --git a/shared/OpenglCodecCommon/GLSharedGroup.cpp b/shared/OpenglCodecCommon/GLSharedGroup.cpp
index 19dfe0f2..e2ed6da3 100755
--- a/shared/OpenglCodecCommon/GLSharedGroup.cpp
+++ b/shared/OpenglCodecCommon/GLSharedGroup.cpp
@@ -28,7 +28,7 @@ static void clearObjectMap(android::DefaultKeyedVector<GLuint, T>& v) {
/**** BufferData ****/
BufferData::BufferData() : m_size(0), m_usage(0), m_mapped(false) {};
-BufferData::BufferData(GLsizeiptr size, void * data) : m_size(size), m_usage(0), m_mapped(false)
+BufferData::BufferData(GLsizeiptr size, const void *data) : m_size(size), m_usage(0), m_mapped(false)
{
void * buffer = NULL;
if (size>0) buffer = m_fixedBuffer.alloc(size);
@@ -260,13 +260,13 @@ SharedTextureDataMap* GLSharedGroup::getTextureData() {
return &m_textureRecs;
}
-void GLSharedGroup::addBufferData(GLuint bufferId, GLsizeiptr size, void * data)
+void GLSharedGroup::addBufferData(GLuint bufferId, GLsizeiptr size, const void* data)
{
android::AutoMutex _lock(m_lock);
m_buffers.add(bufferId, new BufferData(size, data));
}
-void GLSharedGroup::updateBufferData(GLuint bufferId, GLsizeiptr size, void * data)
+void GLSharedGroup::updateBufferData(GLuint bufferId, GLsizeiptr size, const void* data)
{
android::AutoMutex _lock(m_lock);
ssize_t idx = m_buffers.indexOfKey(bufferId);
@@ -304,7 +304,7 @@ bool GLSharedGroup::isBufferMapped(GLuint bufferId) {
return buf->m_mapped;
}
-GLenum GLSharedGroup::subUpdateBufferData(GLuint bufferId, GLintptr offset, GLsizeiptr size, void * data)
+GLenum GLSharedGroup::subUpdateBufferData(GLuint bufferId, GLintptr offset, GLsizeiptr size, const void* data)
{
android::AutoMutex _lock(m_lock);
BufferData * buf = m_buffers.valueFor(bufferId);
diff --git a/shared/OpenglCodecCommon/GLSharedGroup.h b/shared/OpenglCodecCommon/GLSharedGroup.h
index ca338362..2d680be8 100755
--- a/shared/OpenglCodecCommon/GLSharedGroup.h
+++ b/shared/OpenglCodecCommon/GLSharedGroup.h
@@ -46,7 +46,7 @@
struct BufferData {
BufferData();
- BufferData(GLsizeiptr size, void * data);
+ BufferData(GLsizeiptr size, const void* data);
// General buffer state
GLsizeiptr m_size;
@@ -153,13 +153,13 @@ public:
bool isShaderOrProgramObject(GLuint obj);
BufferData * getBufferData(GLuint bufferId);
SharedTextureDataMap* getTextureData();
- void addBufferData(GLuint bufferId, GLsizeiptr size, void * data);
- void updateBufferData(GLuint bufferId, GLsizeiptr size, void * data);
+ void addBufferData(GLuint bufferId, GLsizeiptr size, const void* data);
+ void updateBufferData(GLuint bufferId, GLsizeiptr size, const void* data);
void setBufferUsage(GLuint bufferId, GLenum usage);
void setBufferMapped(GLuint bufferId, bool mapped);
GLenum getBufferUsage(GLuint bufferId);
bool isBufferMapped(GLuint bufferId);
- GLenum subUpdateBufferData(GLuint bufferId, GLintptr offset, GLsizeiptr size, void * data);
+ GLenum subUpdateBufferData(GLuint bufferId, GLintptr offset, GLsizeiptr size, const void* data);
void deleteBufferData(GLuint);
bool isProgram(GLuint program);
diff --git a/system/GLESv2_enc/GL2Encoder.cpp b/system/GLESv2_enc/GL2Encoder.cpp
index cbc0c329..b20c225e 100755
--- a/system/GLESv2_enc/GL2Encoder.cpp
+++ b/system/GLESv2_enc/GL2Encoder.cpp
@@ -536,7 +536,7 @@ void GL2Encoder::s_glBufferData(void * self, GLenum target, GLsizeiptr size, con
SET_ERROR_IF(bufferId==0, GL_INVALID_OPERATION);
SET_ERROR_IF(size<0, GL_INVALID_VALUE);
- ctx->m_shared->updateBufferData(bufferId, size, (void*)data);
+ ctx->m_shared->updateBufferData(bufferId, size, data);
ctx->m_shared->setBufferUsage(bufferId, usage);
ctx->m_glBufferData_enc(self, target, size, data, usage);
}
@@ -549,7 +549,7 @@ void GL2Encoder::s_glBufferSubData(void * self, GLenum target, GLintptr offset,
SET_ERROR_IF(bufferId==0, GL_INVALID_OPERATION);
SET_ERROR_IF(ctx->isBufferTargetMapped(target), GL_INVALID_OPERATION);
- GLenum res = ctx->m_shared->subUpdateBufferData(bufferId, offset, size, (void*)data);
+ GLenum res = ctx->m_shared->subUpdateBufferData(bufferId, offset, size, data);
SET_ERROR_IF(res, res);
ctx->m_glBufferSubData_enc(self, target, offset, size, data);