Mesa (master): meta: Always enable GL_EXT_texture_array in mipmap shader

Ian Romanick idr at kemper.freedesktop.org
Mon Oct 8 03:36:04 UTC 2012


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

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Fri Oct  5 15:17:29 2012 -0700

meta: Always enable GL_EXT_texture_array in mipmap shader

'#extension foo: enable' is harmless.  The functionality is only
actually enabled if the extension is supported.  The shader won't use
the functionality if it's not supported, so we're fine.

Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Brian Paul <brianp at vmware.com>
Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>

---

 src/mesa/drivers/common/meta.c |    9 ++-------
 1 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c
index 8b5cfe4..ba932c7 100644
--- a/src/mesa/drivers/common/meta.c
+++ b/src/mesa/drivers/common/meta.c
@@ -3102,7 +3102,6 @@ setup_glsl_generate_mipmap(struct gl_context *ctx,
 
    if (ctx->API == API_OPENGLES2 || ctx->Const.GLSLVersion < 130) {
       const char *fs_template;
-      const char *extension_mode;
 
       vs_source =
          "attribute vec2 position;\n"
@@ -3114,7 +3113,7 @@ setup_glsl_generate_mipmap(struct gl_context *ctx,
          "   gl_Position = vec4(position, 0.0, 1.0);\n"
          "}\n";
       fs_template =
-         "#extension GL_EXT_texture_array : %s\n"
+         "#extension GL_EXT_texture_array : enable\n"
          "uniform %s texSampler;\n"
          "varying vec3 texCoords;\n"
          "void main()\n"
@@ -3122,12 +3121,8 @@ setup_glsl_generate_mipmap(struct gl_context *ctx,
          "   gl_FragColor = %s(texSampler, %s);\n"
          "}\n";
 
-      extension_mode = ((target == GL_TEXTURE_1D_ARRAY) ||
-                        (target == GL_TEXTURE_2D_ARRAY)) ?
-                       "require" : "disable";
-
       fs_source = ralloc_asprintf(mem_ctx, fs_template,
-                                  extension_mode, sampler->type,
+                                  sampler->type,
                                   sampler->func, sampler->texcoords);
    }
    else {




More information about the mesa-commit mailing list