Re: [spice-common][PATCH 2/2] ppc: Fix quic magic endianess

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, May 19, 2015 at 05:18:44AM -0400, Frediano Ziglio wrote:
> > 
> > ---
> >  common/quic.c | 6 ++++--
> >  1 file changed, 4 insertions(+), 2 deletions(-)
> > 
> > diff --git a/common/quic.c b/common/quic.c
> > index 16290d4..0fbe6c5 100644
> > --- a/common/quic.c
> > +++ b/common/quic.c
> > @@ -23,6 +23,8 @@
> >  #include <config.h>
> >  #endif
> >  
> > +#include <glib.h>
> > +
> >  #include "quic.h"
> >  #include "spice_common.h"
> >  #include "bitops.h"
> > @@ -1277,7 +1279,7 @@ int quic_encode(QuicContext *quic, QuicImageType type,
> > int width, int height,
> >      encoder->io_word = 0;
> >      encoder->io_available_bits = 32;
> >  
> > -    encode_32(encoder, QUIC_MAGIC);
> > +    encode_32(encoder, GUINT32_TO_LE(QUIC_MAGIC));
> >      encode_32(encoder, QUIC_VERSION);
> >      encode_32(encoder, type);
> >      encode_32(encoder, width);
> 
> Why don't you define QUIC_MAGIC as a constant like
> 
> /* ASCII "QUIC" */
> #define QUIC_MAGIC 0x43495551
> 
> instead ?

Yes, that's probably better given that we can't keep that self-contained
(ie something similar would be needed in spice-gtk).

Christophe

Attachment: pgpdWURamcbDp.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]