diff options
author | Yahan Zhou <yahan@google.com> | 2016-08-10 21:15:55 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2016-08-10 21:15:55 +0000 |
commit | 3c108183f38542402daed9c40e011d55520d6f71 (patch) | |
tree | 3b5a5a174c7ebc488136c9a93f9ebb4a1f451fc7 | |
parent | 1fb7ea129dc0538080e1712adae34cf91ba1c962 (diff) | |
parent | ae30fe8405b0fb6fe459a0b9cac5111577f12ade (diff) | |
download | goldfish-opengl-3c108183f38542402daed9c40e011d55520d6f71.tar.gz |
Merge "Revert "glGetProgramiv validation on the guest""
-rwxr-xr-x | system/GLESv2_enc/GL2Encoder.cpp | 29 | ||||
-rw-r--r-- | system/GLESv2_enc/GL2Encoder.h | 4 |
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; |