On 3 October 2015 at 02:12, Marek Olšák <mar...@gmail.com> wrote:
> I'm not sure if this is correct or if we should just return NULL in
> this case like the "case" statement above that does.
>
Actually I was thinking about bailing out when the requested attribute
is set. I.e.
diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
index 1740ee3..0450269 100644
--- a/src/egl/drivers/dri2/egl_dri2.c
+++ b/src/egl/drivers/dri2/egl_dri2.c
@@ -237,6 +237,8 @@ dri2_add_config(_EGLDisplay *disp, const
__DRIconfig *dri_config, int id,
case __DRI_ATTRIB_FRAMEBUFFER_SRGB_CAPABLE:
srgb = value != 0;
+ if (!dpy->Extensions.KHR_gl_colorspace && srgb)
+ return NULL;
break;
default:
-Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
