On Thu, 5 May 2011 12:39:57 -0700, Jesse Barnes <jbar...@virtuousgeek.org> 
wrote:
> Just add a new struct to remain compatible with existing code.
> 
> Signed-off-by: Jesse Barnes <jbar...@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;

Attachment: pgpGoE7Hp2AZA.pgp
Description: PGP signature

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to