On Mon, May 18, 2015 at 6:13 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote: > On Mon, May 18, 2015 at 03:10:57PM +0200, Fabiano Fidêncio wrote: >> Since commit d39dfbfe lz magic is always treated as LE when encoded. >> Treating it as a LE when decoding is now necessary as well. > > Thanks for the catch. The issue is that given the way LZ_MAGIC is > defined in lz_common.h, it will have a different value when running on a > big-endian or on a little-endian system. decode_32/encode_32 try to > handle endianness on their own, but in the case of LZ_MAGIC, we don't > want this special treatment to happen, so we need an extra endian swap. > I assume this is what you mean when you say we want to treat it as LE in > the log? Yep, would be better to improve the commit log? > > Christophe > > _______________________________________________ > Spice-devel mailing list > Spice-devel@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/spice-devel > Best Regards, -- Fabiano Fidêncio _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel