Commit d7460c7e by Jamie Madill

Fix gl::ValidTextureTarget to work on all enum values.

Change-Id: I347583ae1adcaaae9feeb1276f6150ceb8739885 Reviewed-on: https://chromium-review.googlesource.com/181557Reviewed-by: 's avatarGeoff Lang <geofflang@chromium.org> Tested-by: 's avatarJamie Madill <jmadill@chromium.org>
parent 1fc7e2c5
...@@ -23,15 +23,19 @@ namespace gl ...@@ -23,15 +23,19 @@ namespace gl
bool ValidTextureTarget(const Context *context, GLenum target) bool ValidTextureTarget(const Context *context, GLenum target)
{ {
if (context->getClientVersion() < 3) switch (target)
{
if (target == GL_TEXTURE_3D || target == GL_TEXTURE_2D_ARRAY)
{ {
case GL_TEXTURE_2D:
case GL_TEXTURE_CUBE_MAP:
return true;
case GL_TEXTURE_3D:
case GL_TEXTURE_2D_ARRAY:
return (context->getClientVersion() >= 3);
default:
return false; return false;
} }
}
return true;
} }
bool ValidFramebufferTarget(GLenum target) bool ValidFramebufferTarget(GLenum target)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment