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.
Marek On Sat, Oct 3, 2015 at 12:48 AM, Emil Velikov <emil.l.veli...@gmail.com> wrote: > Otherwise the user has no way of using it, and we'll try to access the > linear one. > > Cc: Mauro Rossi <issor.or...@gmail.com> > Cc: Chih-Wei Huang <cwhu...@android-x86.org> > Cc: "11.0" <mesa-sta...@lists.freedesktop.org> > Fixes: c2c2e9ab604(egl: implement EGL_KHR_gl_colorspace (v2)) > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91596 > Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> > --- > src/egl/drivers/dri2/egl_dri2.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c > index 1740ee3..049bebb 100644 > --- a/src/egl/drivers/dri2/egl_dri2.c > +++ b/src/egl/drivers/dri2/egl_dri2.c > @@ -236,7 +236,8 @@ dri2_add_config(_EGLDisplay *disp, const __DRIconfig > *dri_config, int id, > break; > > case __DRI_ATTRIB_FRAMEBUFFER_SRGB_CAPABLE: > - srgb = value != 0; > + if (dpy->Extensions.KHR_gl_colorspace) > + srgb = value != 0; > break; > > default: > -- > 2.5.0 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
