On 3 September 2018 at 19:14, Eric Engestrom <eric.engest...@intel.com> wrote: > On Monday, 2018-09-03 13:05:22 +0100, Emil Velikov wrote: >> From: Emil Velikov <emil.veli...@collabora.com> >> >> As the spec says, the function is a no-op when the surface is not a >> window one. >> >> That spec implies that EGL_TRUE should be returned in that case, yet >> the ARM driver seems to return EGL_FALSE + EGL_BAD_SURFACE. >> >> The Nvidia driver returns EGL_TRUE. We follow that behaviour until a >> decision is made. >> >> https://gitlab.khronos.org/egl/API/merge_requests/17 >> >> Cc: samiuddi <sami.uddin.moham...@intel.com> >> Cc: Eric Engestrom <eric.engest...@intel.com> >> Cc: Erik Faye-Lund <kusmab...@gmail.com> >> Cc: Tomasz Figa <tf...@chromium.org> >> Cc: <mesa-sta...@lists.freedesktop.org> >> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> >> --- >> Since this is a high-level API decision I've moved it to eglapi.c >> This will allow us to avoid duplicating the check across each platform >> codebase ... or more crashes because we forgot to update one. > > Very good point! Series is: > Reviewed-by: Eric Engestrom <eric.engest...@intel.com> > > Could you add the references to the bugs and tests fixed from the other > patch, and add Cc: stable to this patch and the eglSwapBuffers() one > (the rest of the series is just cleanup of stuff that these two patches > makes unnecessary, so let's just not touch that on stable branches)? > Based on the earlier patches CtsDisplayTestCases will fail (instead of crash) since it expects EGL_FALSE. Cannot see any references or bugs - am I having a blind moment?
Thanks Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
