From: Erik Faye-Lund Date: Mon, 25 Feb 2019 11:06:23 +0000 (+0100) Subject: mesa/main: make the TEXTURE_CUBE_MAP checks consistent X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=147751a856432157726d0462406f89b0f87aa8d9;p=mesa.git mesa/main: make the TEXTURE_CUBE_MAP checks consistent IsEnabled(TEXTURE_CUBE_MAP) isn't supposed to be allowed, but our check allowed this anyway. Let's make these checks consistent, and while we're at it, modernize them a bit. Signed-off-by: Erik Faye-Lund Reviewed-by: Ian Romanick --- diff --git a/src/mesa/main/enable.c b/src/mesa/main/enable.c index 83e7f9734d1..09f33afbc57 100644 --- a/src/mesa/main/enable.c +++ b/src/mesa/main/enable.c @@ -819,9 +819,9 @@ _mesa_set_enable(struct gl_context *ctx, GLenum cap, GLboolean state) /* GL_ARB_texture_cube_map */ case GL_TEXTURE_CUBE_MAP: - if (ctx->API != API_OPENGL_COMPAT && ctx->API != API_OPENGLES) + if (!_mesa_has_ARB_texture_cube_map(ctx) && + !_mesa_has_OES_texture_cube_map(ctx)) goto invalid_enum_error; - CHECK_EXTENSION(ARB_texture_cube_map); if (!enable_texture(ctx, state, TEXTURE_CUBE_BIT)) { return; } @@ -1595,7 +1595,9 @@ _mesa_IsEnabled( GLenum cap ) /* GL_ARB_texture_cube_map */ case GL_TEXTURE_CUBE_MAP: - CHECK_EXTENSION(ARB_texture_cube_map); + if (!_mesa_has_ARB_texture_cube_map(ctx) && + !_mesa_has_OES_texture_cube_map(ctx)) + goto invalid_enum_error; return is_texture_enabled(ctx, TEXTURE_CUBE_BIT); /* GL_EXT_secondary_color */