2009/12/28 Michel Dänzer <[email protected]>: > On Mon, 2009-12-28 at 13:51 +0100, Maarten Maathuis wrote: >> 2009/12/28 Michel Dänzer <[email protected]>: >> > From: Thomas Hellstrom <[email protected]> >> > >> > [ Michel: Minor fixups to address compiler warnings ] >> > >> > Signed-off-by: Thomas Hellstrom <[email protected]> >> > Signed-off-by: Michel Dänzer <[email protected]> >> > --- >> > exa/exa_priv.h | 10 +++ >> > exa/exa_unaccel.c | 231 >> > +++++++++++++++++++++++++++++++++++++++++++---------- >> > 2 files changed, 200 insertions(+), 41 deletions(-) >> > >> > diff --git a/exa/exa_priv.h b/exa/exa_priv.h >> > index c559b9a..21d9646 100644 >> > --- a/exa/exa_priv.h >> > +++ b/exa/exa_priv.h >> > @@ -165,6 +165,7 @@ typedef struct { >> > BitmapToRegionProcPtr SavedBitmapToRegion; >> > CreateScreenResourcesProcPtr SavedCreateScreenResources; >> > ModifyPixmapHeaderProcPtr SavedModifyPixmapHeader; >> > + SourceValidateProcPtr SavedSourceValidate; >> > #ifdef RENDER >> > CompositeProcPtr SavedComposite; >> > TrianglesProcPtr SavedTriangles; >> >> I don't quite get why this has to be in ExaScreenPrivRec, it seems to >> be only used in one function. > > It allows using the swap() macro. I agree it's not strictly necessary, > but frankly I don't see the big deal.
I meant the stuff below, sorry if that was unclear. > > > -- > Earthling Michel Dänzer | http://www.vmware.com > Libre software enthusiast | Debian, X and DRI developer > _______________________________________________ xorg-devel mailing list [email protected] http://lists.x.org/mailman/listinfo/xorg-devel
