aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYahan Zhou <yahan@google.com>2016-08-10 21:15:55 +0000
committerAndroid (Google) Code Review <android-gerrit@google.com>2016-08-10 21:15:55 +0000
commit3c108183f38542402daed9c40e011d55520d6f71 (patch)
tree3b5a5a174c7ebc488136c9a93f9ebb4a1f451fc7
parent1fb7ea129dc0538080e1712adae34cf91ba1c962 (diff)
parentae30fe8405b0fb6fe459a0b9cac5111577f12ade (diff)
downloadgoldfish-opengl-3c108183f38542402daed9c40e011d55520d6f71.tar.gz
Merge "Revert "glGetProgramiv validation on the guest""
-rwxr-xr-xsystem/GLESv2_enc/GL2Encoder.cpp29
-rw-r--r--system/GLESv2_enc/GL2Encoder.h4
2 files changed, 1 insertions, 32 deletions
diff --git a/system/GLESv2_enc/GL2Encoder.cpp b/system/GLESv2_enc/GL2Encoder.cpp
index 26cf6eec..97557c98 100755
--- a/system/GLESv2_enc/GL2Encoder.cpp
+++ b/system/GLESv2_enc/GL2Encoder.cpp
@@ -99,7 +99,6 @@ GL2Encoder::GL2Encoder(IOStream *stream, ChecksumCalculator *protocol)
OVERRIDE(glGetShaderSource);
OVERRIDE(glGetShaderInfoLog);
OVERRIDE(glGetProgramInfoLog);
- OVERRIDE(glGetProgramiv);
OVERRIDE(glGetUniformLocation);
OVERRIDE(glUseProgram);
@@ -167,6 +166,7 @@ GLenum GL2Encoder::s_glGetError(void * self)
}
return ctx->m_glGetError_enc(self);
+
}
void GL2Encoder::s_glFlush(void *self)
@@ -999,33 +999,6 @@ void GL2Encoder::s_glGetProgramInfoLog(void *self, GLuint program, GLsizei bufsi
ctx->m_glGetProgramInfoLog_enc(self, program, bufsize, length, infolog);
}
-void GL2Encoder::s_glGetProgramiv(void *self, GLuint program, GLenum pname,
- GLint* params)
-{
- // We need to do the validation here because the spec says if an error is
- // generated, no change is made to the contents of params. Our encoder does
- // not respect this.
- GL2Encoder *ctx = (GL2Encoder*)self;
- switch (pname) {
- case GL_DELETE_STATUS:
- case GL_LINK_STATUS:
- case GL_VALIDATE_STATUS:
- case GL_INFO_LOG_LENGTH:
- case GL_ATTACHED_SHADERS:
- case GL_ACTIVE_ATTRIBUTES:
- case GL_ACTIVE_ATTRIBUTE_MAX_LENGTH:
- case GL_ACTIVE_UNIFORMS:
- case GL_ACTIVE_UNIFORM_MAX_LENGTH:
- break;
- default:
- SET_ERROR_IF(1, GL_INVALID_ENUM);
- break;
- }
- SET_ERROR_IF(!ctx->m_shared->isObject(program), GL_INVALID_VALUE);
- SET_ERROR_IF(!ctx->m_shared->isProgram(program), GL_INVALID_OPERATION);
- ctx->m_glGetProgramiv_enc(self, program, pname, params);
-}
-
void GL2Encoder::s_glDeleteShader(void *self, GLenum shader)
{
GL2Encoder *ctx = (GL2Encoder*)self;
diff --git a/system/GLESv2_enc/GL2Encoder.h b/system/GLESv2_enc/GL2Encoder.h
index 5bdbd71a..69bd4cf9 100644
--- a/system/GLESv2_enc/GL2Encoder.h
+++ b/system/GLESv2_enc/GL2Encoder.h
@@ -174,10 +174,6 @@ private:
static void s_glGetProgramInfoLog(void *self,GLuint program,
GLsizei bufsize, GLsizei* length, GLchar* infolog);
- glGetProgramiv_client_proc_t m_glGetProgramiv_enc;
- static void s_glGetProgramiv(void *self, GLuint program, GLenum pname,
- GLint* params);
-
glGetUniformLocation_client_proc_t m_glGetUniformLocation_enc;
static int s_glGetUniformLocation(void *self, GLuint program, const GLchar *name);
glUseProgram_client_proc_t m_glUseProgram_enc;