Re: [PATCH] dix/glx/composite: consolidate visual resize in one place.

2009-10-06 Thread Aaron Plattner
On Tue, Sep 29, 2009 at 04:19:06PM -0700, Julien Cristau wrote: > On Wed, Sep 30, 2009 at 09:06:42 +1000, Peter Hutterer wrote: > > On Tue, Sep 29, 2009 at 11:49:09AM +1000, Dave Airlie wrote: > > > diff --git a/include/colormap.h b/include/colormap.h > > > index a3467c9..eb0f670 100644 > > > --- a

Re: [PATCH] dix/glx/composite: consolidate visual resize in one place.

2009-09-29 Thread Julien Cristau
On Wed, Sep 30, 2009 at 09:06:42 +1000, Peter Hutterer wrote: > On Tue, Sep 29, 2009 at 11:49:09AM +1000, Dave Airlie wrote: > > diff --git a/include/colormap.h b/include/colormap.h > > index a3467c9..eb0f670 100644 > > --- a/include/colormap.h > > +++ b/include/colormap.h > > @@ -179,4 +179,9 @@

Re: [PATCH] dix/glx/composite: consolidate visual resize in one place.

2009-09-29 Thread Peter Hutterer
On Tue, Sep 29, 2009 at 11:49:09AM +1000, Dave Airlie wrote: > The previous code was copied and in both cases incorrectly fixed > up the colormaps after resizing the visuals, this patch consolidates > the visual resize + colormaps fixups in one place. This version > also consolidates the vid alloca

Re: [PATCH] dix/glx/composite: consolidate visual resize in one place.

2009-09-28 Thread Keith Packard
Excerpts from Dave Airlie's message of Mon Sep 28 18:49:09 -0700 2009: > The previous code was copied and in both cases incorrectly fixed > up the colormaps after resizing the visuals, this patch consolidates > the visual resize + colormaps fixups in one place. This version > also consolidates the

[PATCH] dix/glx/composite: consolidate visual resize in one place.

2009-09-28 Thread Dave Airlie
The previous code was copied and in both cases incorrectly fixed up the colormaps after resizing the visuals, this patch consolidates the visual resize + colormaps fixups in one place. This version also consolidates the vid allocation for the DepthPtr inside the function. I'm not 100% sure colorma

Re: [PATCH] dix/glx/composite: consolidate visual resize in one place.

2009-09-28 Thread Keith Packard
Excerpts from Dave Airlie's message of Mon Sep 28 16:30:45 -0700 2009: > The previous code was copied and in both cases incorrectly fixed > up the colormaps after resizing the visuals, this patch consolidates > the visual resize + colormaps fixups in one place. Looks like this function should also

[PATCH] dix/glx/composite: consolidate visual resize in one place.

2009-09-28 Thread Dave Airlie
The previous code was copied and in both cases incorrectly fixed up the colormaps after resizing the visuals, this patch consolidates the visual resize + colormaps fixups in one place. I'm not 100% sure colormap.[ch] is the correct place for this but visuals are mostly created in fb and I know tha