On 18/01/17 10:38, Alejandro Piñeiro wrote: > Reviewed by: Alejandro Piñeiro <[email protected]> typo (although I saw several "Reviewed by" on the list, I think that the official is "Reviewed-by", and I guess that is what patchwork catches) Reviewed-by: Alejandro Piñeiro <[email protected]>
> > On 18/01/17 06:16, Nicolai Hähnle wrote: >> From: Nicolai Hähnle <[email protected]> >> >> 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 <[email protected]> >> --- >> 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 [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
