Module: Mesa
Branch: master
Commit: 147751a856432157726d0462406f89b0f87aa8d9
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=147751a856432157726d0462406f89b0f87aa8d9

Author: Erik Faye-Lund <[email protected]>
Date:   Mon Feb 25 12:06:23 2019 +0100

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 <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>

---

 src/mesa/main/enable.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

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 */

_______________________________________________
mesa-commit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to