On Tue, 10 May 2011 11:59:56 -0700 Eric Anholt <e...@anholt.net> wrote:
> 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. Ok, wanna push your existing patch for that or should I push it with this stuff? Thanks, -- Jesse Barnes, Intel Open Source Technology Center _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
