On 28 September 2016 at 07:28, Chad Versace <chadvers...@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 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
