[Mesa-dev] [PATCH] egl_glx: use casts to silence warnings in GLX_eglDestroyContext()

Chia-I Wu olvaffe at gmail.com
Fri Dec 2 19:07:22 PST 2011


On Fri, Dec 2, 2011 at 10:53 PM, Brian Paul <brianp at vmware.com> wrote:
> ---
>  src/egl/drivers/glx/egl_glx.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/egl/drivers/glx/egl_glx.c b/src/egl/drivers/glx/egl_glx.c
> index bdc8a28..6479daf 100644
> --- a/src/egl/drivers/glx/egl_glx.c
> +++ b/src/egl/drivers/glx/egl_glx.c
> @@ -724,7 +724,7 @@ GLX_eglDestroyContext(_EGLDriver *drv, _EGLDisplay *disp, _EGLContext *ctx)
>
>    if (_eglPutContext(ctx)) {
>       assert(GLX_ctx);
> -      GLX_drv->glXDestroyContext(disp, ctx);
> +      GLX_drv->glXDestroyContext((Display *) disp, (GLXContext) ctx);
This warning points out a real issue:  Display and GLXContext should
be passed to glXDestroyContext, not _EGLDisplay and _EGLContext.  I
will commit a fix shortly.

>
>       free(GLX_ctx);
>    }
> --
> 1.7.3.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev



-- 
olv at LunarG.com


More information about the mesa-dev mailing list