From: Ian Romanick Date: Wed, 7 Jun 2017 08:35:08 +0000 (-0700) Subject: mesa: GL_TEXTURE_BORDER_COLOR exists in OpenGL 1.0, so don't depend on GL_ARB_texture... X-Git-Tag: upstream/18.1.0~8211 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2e3f40272e7d8905969e7dd319a463097787fb57;p=platform%2Fupstream%2Fmesa.git mesa: GL_TEXTURE_BORDER_COLOR exists in OpenGL 1.0, so don't depend on GL_ARB_texture_border_clamp On NV20 (and probably also on earlier NV GPUs that lack GL_ARB_texture_border_clamp) fixes the following piglit tests: gl-1.0-beginend-coverage gltexparameter[if]{v,} push-pop-texture-state texwrap 1d texwrap 1d proj texwrap 2d proj texwrap formats All told, 49 more tests pass on NV20 (10de:0201). No changes on Intel CI run or RV250 (1002:4c66). Signed-off-by: Ian Romanick Reviewed-by: Ilia Mirkin --- diff --git a/src/mesa/main/texparam.c b/src/mesa/main/texparam.c index 3c110de..857faf6 100644 --- a/src/mesa/main/texparam.c +++ b/src/mesa/main/texparam.c @@ -736,8 +736,16 @@ set_tex_parameterf(struct gl_context *ctx, break; case GL_TEXTURE_BORDER_COLOR: + /* Border color exists in desktop OpenGL since 1.0 for GL_CLAMP. In + * OpenGL ES 2.0+, it only exists in when GL_OES_texture_border_clamp is + * enabled. It is never available in OpenGL ES 1.x. + * + * FIXME: Every driver that supports GLES2 has this extension. Elide + * the check? + */ if (ctx->API == API_OPENGLES || - !ctx->Extensions.ARB_texture_border_clamp) + (ctx->API == API_OPENGLES2 && + !ctx->Extensions.ARB_texture_border_clamp)) goto invalid_pname; if (!_mesa_target_allows_setting_sampler_parameters(texObj->Target))