[PATCH 2/2] render: Fix byteswapping of gradient stops

Keith Packard keithp at keithp.com
Thu Dec 2 07:41:15 PST 2010


On Wed, 1 Dec 2010 23:59:04 +0100, Julien Cristau <jcristau at debian.org> wrote:
> On Thu, Nov  4, 2010 at 02:44:48 +0100, Soeren Sandmann wrote:
> > Andrea Canciani <ranma42 at gmail.com> writes:

> > >      }
> > >      colors = (CARD16 *)(stops);
> > >      for (i = 0; i < 4*num; ++i) {
> > > -        swaps(stops, n);
> > > -        ++stops;
> > > +        swaps(colors, n);
> > > +        ++colors;
> > >      }
> > >  }
> > 
> > Reviewed-by: Soren Sandmann <sandmann at daimi.au.dk>
> > 
> Reviewed-by: Julien Cristau <jcristau at debian.org>
> 
> Keith, can you take this one?  It would be nice to have it on the stable
> branches.

Merged.
   279ef1f..dab064f  master -> master

Note that I have *not* merged the patch which allows for single-stop
gradients -- that awaits autoconf changes to require a fixed version of
pixman. Soren's question about whether the Render protocol version
should be bumped is also valid, but I think that would require a change
to the protocol package as well as the X server.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20101202/c6dc5e75/attachment.pgp>


More information about the xorg-devel mailing list