[Mesa-dev] [PATCH 7/7] egl: Unify the EGLint/EGLAttrib paths in eglCreateSync*

Emil Velikov emil.l.velikov at gmail.com
Thu Sep 29 17:05:47 UTC 2016


On 28 September 2016 at 07:28, Chad Versace <chadversary at chromium.org> wrote:
> Pre-patch, there were two code paths for parsing EGLSync attribute
> lists: one path for old-style EGLint lists, used by eglCreateSyncKHR,
> and another for new-style EGLAttrib lists, used by eglCreateSync (1.5)
> and eglCreateSync64 (EGL_KHR_cl_event2).
>
Actually we might want to use the same helper instead of
_eglConvertAttribsToInt for all entry points where the pre-1.5 entry
point was using EGLint while the EGL 1.5 one uses EGLAttrib.

In those cases we currently a) loose the upper bits (admittedly they
aren't used afaics) and b) we'll error out if the user provides an
empty/null list (not the most useful thing to do, but still).

Afaics all the EGL 1.5 entry points either explicitly state that
attrib_list can be null or empty or say that all the unspecified
attribs will be set to their default value.

Regards,
Emil


More information about the mesa-dev mailing list