Reviewed by: Alejandro Piñeiro <apinhe...@igalia.com>
On 18/01/17 06:16, Nicolai Hähnle wrote:
> From: Nicolai Hähnle <nicolai.haeh...@amd.com>
>
> Since _mesa_ClampColor properly checks for support of the API function
> now, it's meta callers need to check support as well.
>
> Fixes: 963311b71f ("mesa/main: fix version/extension checks in
> _mesa_ClampColor")
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99401
> Tested-by: Mark Janes <mark.a.ja...@intel.com>
> ---
> src/mesa/main/attrib.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c
> index 9c7f5ed..ada2203 100644
> --- a/src/mesa/main/attrib.c
> +++ b/src/mesa/main/attrib.c
> @@ -1071,7 +1071,8 @@ _mesa_PopAttrib(void)
> if (ctx->Extensions.ARB_color_buffer_float)
> _mesa_ClampColor(GL_CLAMP_FRAGMENT_COLOR_ARB,
> color->ClampFragmentColor);
> - _mesa_ClampColor(GL_CLAMP_READ_COLOR_ARB,
> color->ClampReadColor);
> + if (ctx->Extensions.ARB_color_buffer_float || ctx->Version >=
> 30)
> + _mesa_ClampColor(GL_CLAMP_READ_COLOR_ARB,
> color->ClampReadColor);
>
> /* GL_ARB_framebuffer_sRGB / GL_EXT_framebuffer_sRGB */
> if (ctx->Extensions.EXT_framebuffer_sRGB)
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
