meta: Always enable GL_EXT_texture_array in mipmap shader
authorIan Romanick <ian.d.romanick@intel.com>
Fri, 5 Oct 2012 22:17:29 +0000 (15:17 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Mon, 8 Oct 2012 03:35:47 +0000 (20:35 -0700)
'#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@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Brian Paul <brianp@vmware.com>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
src/mesa/drivers/common/meta.c

index 8b5cfe44ab31a2f622a7bee332b0b390ee420a96..ba932c78ba6e94bba82551f5340401240b278004 100644 (file)
@@ -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 {