[Mesa-dev] [PATCH] glx/dri2: use GLX prefix to enable INTEL_swap_event

Ian Romanick idr at freedesktop.org
Thu Feb 9 14:19:43 PST 2012


On 02/08/2012 11:49 AM, Robert Bragg wrote:
> This adds the "GLX_" prefix to the string we pass to
> __glXEnableDirectExtension() otherwise it doesn't match the name we have
> in known_glx_extensions[] in glxextensions.c and doesn't get enabled.
>
> This mistake wasn't noticed before since GLX_INTEL_swap_event was being
> implicitly enabled for GLX v1.4 but that was changed in commit
> d3f7597bc9f6d5
>
> Signed-off-by: Robert Bragg<robert at sixbynine.org>

Oops.  Nice catch.

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

> ---
>   src/glx/dri2_glx.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/glx/dri2_glx.c b/src/glx/dri2_glx.c
> index 10b6f52..1fd8c99 100644
> --- a/src/glx/dri2_glx.c
> +++ b/src/glx/dri2_glx.c
> @@ -950,7 +950,7 @@ dri2BindExtensions(struct dri2_screen *psc, const __DRIextension **extensions)
>      __glXEnableDirectExtension(&psc->base, "GLX_SGI_make_current_read");
>
>      /* FIXME: if DRI2 version supports it... */
> -   __glXEnableDirectExtension(&psc->base, "INTEL_swap_event");
> +   __glXEnableDirectExtension(&psc->base, "GLX_INTEL_swap_event");
>
>      if (psc->dri2->base.version>= 3) {
>         const unsigned mask = psc->dri2->getAPIMask(psc->driScreen);


More information about the mesa-dev mailing list