On 3 September 2018 at 19:14, Eric Engestrom <[email protected]> wrote: > On Monday, 2018-09-03 13:05:22 +0100, Emil Velikov wrote: >> From: Emil Velikov <[email protected]> >> >> 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 <[email protected]> >> Cc: Eric Engestrom <[email protected]> >> Cc: Erik Faye-Lund <[email protected]> >> Cc: Tomasz Figa <[email protected]> >> Cc: <[email protected]> >> Signed-off-by: Emil Velikov <[email protected]> >> --- >> 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 <[email protected]> > > 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 [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
