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.


-- 
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

Reply via email to