On 06/01/2013 06:23 AM, Marek Olšák wrote:
---
  src/glsl/glcpp/glcpp-parse.y           |   14 +++++++-------
  src/mesa/drivers/common/meta.c         |    3 ++-
  src/mesa/main/ff_fragment_shader.cpp   |    3 ++-
  src/mesa/main/teximage.c               |    2 +-
  src/mesa/state_tracker/st_extensions.c |    3 +--
  5 files changed, 13 insertions(+), 12 deletions(-)


diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index 9aaa63f..1c27bef 100644
--- a/src/mesa/main/teximage.c
+++ b/src/mesa/main/teximage.c
@@ -798,7 +798,7 @@ _mesa_select_tex_object(struct gl_context *ctx,
                  ctx->Extensions.ARB_texture_buffer_object ?
                  texUnit->CurrentTex[TEXTURE_BUFFER_INDEX] : NULL;
        case GL_TEXTURE_EXTERNAL_OES:
-         return ctx->Extensions.OES_EGL_image_external
+         return _mesa_is_gles(ctx) && ctx->Extensions.OES_EGL_image_external
              ? texUnit->CurrentTex[TEXTURE_EXTERNAL_INDEX] : NULL;
        case GL_TEXTURE_2D_MULTISAMPLE:
           return ctx->Extensions.ARB_texture_multisample

A similar check is needed in the switch statement at top of
teximage.c:_mesa_EGLImageTargetTexture2DOES. Otherwise, GL_INVALID_ENUM
does not get emitted on non-GLES contexts.

With that change, this is
Reviewed-by: Chad Versace <chad.vers...@linux.intel.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to