Hi,<br><br>On Monday, May 11, 2015, Bryce Harrington <<a href="mailto:bryce@osg.samsung.com">bryce@osg.samsung.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, May 09, 2015 at 02:56:54PM +0100, Daniel Stone wrote:<br>
> On 7 May 2015 at 01:44, Bryce Harrington <<a href="javascript:;" onclick="_e(event, 'cvml', 'bryce@osg.samsung.com')">bryce@osg.samsung.com</a>> wrote:<br>
> > +static void<br>
> > +copy_bgra_yflip(uint8_t *dst, uint8_t *src, int height, int stride)<br>
> > +<br>
> > +static void<br>
> > +copy_bgra(uint8_t *dst, uint8_t *src, int height, int stride)<br>
> > +<br>
> > +static void<br>
> > +copy_row_swap_RB(void *vdst, void *vsrc, int bytes)<br>
> > +<br>
> > +static void<br>
> > +copy_rgba_yflip(uint8_t *dst, uint8_t *src, int height, int stride)<br>
> > +<br>
> > +static void<br>
> > +copy_rgba(uint8_t *dst, uint8_t *src, int height, int stride)<br>
><br>
> Can we please use Pixman for these instead of open-coding them?<br>
<br>
I don't understand, explain?  These routines already existing in the<br>
screenshooter client, I just moved them to shared so I could reuse them.</blockquote><div><br></div><div>Ugh, I'd missed that. Fair enough then, but could you please add a /* XXX: It would be nice if we used Pixman to do all this rather than our own implementation */?</div><div><br></div><div>Cheers,</div><div>Daniel<span></span></div>