[PATCH] glproto: add a new GLXBufferSwapComplete struct that matches the spec

Eric Anholt eric at anholt.net
Tue May 10 11:59:56 PDT 2011


On Thu, 5 May 2011 12:39:57 -0700, Jesse Barnes <jbarnes at virtuousgeek.org> wrote:
> Just add a new struct to remain compatible with existing code.
> 
> Signed-off-by: Jesse Barnes <jbarnes at virtuousgeek.org>
> 
> diff --git a/configure.ac b/configure.ac
> index a3047e4..a6c301c 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1,5 +1,5 @@
>  AC_PREREQ([2.60])
> -AC_INIT([GLProto], [1.4.13], [https://bugs.freedesktop.org/enter_bug.cgi?product=xorg])
> +AC_INIT([GLProto], [1.4.14], [https://bugs.freedesktop.org/enter_bug.cgi?product=xorg])
>  AM_INIT_AUTOMAKE([foreign dist-bzip2])
>  AM_MAINTAINER_MODE
>  
> diff --git a/glxproto.h b/glxproto.h
> index dfa0647..3f9e837 100644
> --- a/glxproto.h
> +++ b/glxproto.h
> @@ -1375,6 +1375,20 @@ typedef struct {
>      BYTE pad;
>      CARD16 sequenceNumber B16;
>      CARD16 event_type B16;

While this is the compat structure, I'd still like to see the padding
explicit so I don't worry about it when reading the code ever again.

> +    CARD32 drawable;
> +    CARD32 ust_hi B32;
> +    CARD32 ust_lo B32;
> +    CARD32 msc_hi B32;
> +    CARD32 msc_lo B32;
> +    CARD32 sbc_hi B32;
> +    CARD32 sbc_lo B32;
> +} xGLXBufferSwapComplete;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20110510/99af6bfe/attachment.pgp>


More information about the xorg-devel mailing list