[Mesa-dev] [PATCH 5/6] meta: Always enable GL_EXT_texture_array in mipmap shader

Ian Romanick idr at freedesktop.org
Fri Oct 5 15:56:51 PDT 2012


From: Ian Romanick <ian.d.romanick at intel.com>

'#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>
---
 src/mesa/drivers/common/meta.c | 9 ++-------
 1 file 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 {
-- 
1.7.11.4



More information about the mesa-dev mailing list