egl: use _eglError's 'msg' as an actual message in EGL_KHR_debug
authorEmil Velikov <emil.velikov@collabora.com>
Thu, 7 Sep 2017 16:03:52 +0000 (17:03 +0100)
committerEmil Velikov <emil.l.velikov@gmail.com>
Tue, 19 Sep 2017 18:07:12 +0000 (19:07 +0100)
Seemingly, the original intent behind _eglError's 'msg' was aimed to
provide a function name.

At some point, people started using it the way EGL_KHR_debug's
callback() message is meant to be used. Aka providing meaningful
information to the developer/user.

Swap the funcName/msg argument order in the _eglDebugReport() call.
The 'funcName' variable is implicitly set, props to the
_eglSetFuncName() call at the start of each public entrypoint.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>
src/egl/main/eglcurrent.c

index 26f42761e26c04752c1382078c651256d6be208d..df0463d70e5a35716257dc466c4206cadb7014f9 100644 (file)
@@ -287,7 +287,7 @@ _eglError(EGLint errCode, const char *msg)
          type = EGL_DEBUG_MSG_ERROR_KHR;
       }
 
-      _eglDebugReport(errCode, msg, type, NULL);
+      _eglDebugReport(errCode, NULL, type, msg);
    } else
       _eglInternalError(errCode, msg);