On 12 September 2016 at 23:19, Adam Jackson <a...@redhat.com> wrote:

> +static EGLBoolean
> +_eglSetFuncName(const char *funcName, _EGLDisplay *disp, EGLenum objectType, 
> _EGLResource *object)
> +{

> +         if (disp != NULL) {
> +            thr->CurrentObjectLabel = disp->Label;
> +         }
> +      } else {
> +         /*
> +          * Everything else will either be NULL or a valid _EGLResource
> +          * pointer.
> +          */
> +         if (object != NULL) {
First thought was object/disp can never be NULL, only to remember that
input validation happens at a later stage. Oh well :-]

Going through table 13.2 and the below there are some discrepancies.
AFAICT some of those can be seen as spec bugs (B), while others seem
to be missing (M)
 - thread - eglBindAPI (M)
 - display - eglGetCurrentDisplay (B)
 - context - eglQueryAPI (M), eglGetCurrentContext (B)
 - surface - eglSwapInterval (B), eglGetCurrentSurface(B)
 - image - ok
 - sync - ok


-Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to