On Wed, Aug 02, 2017 at 05:10:56AM -0400, Frediano Ziglio wrote: > > > > The naming is odd as this is just an alias for channel->correlate_row. > > This will also help in subsequent commits to make things more > > consistent with quic_rgb_tmpl.c > > > > Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> > > > This IMHO should go as 1/13 and is more related to previous series. The patch by itself is not really correct as in - decorrelate_drow[0] = family.xlatU2L[cur_row->a]; xsalU2L[] is initialized by decorrelate_init(), so decorrelate_row is not a bad name. Imo this patch only makes sense in the context of unifying quic_tmpl.c and quic_rgb_tmpl.c, so I would not move it out of the series. Christophe
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel