[Spice-devel] [spice-protocol PATCH v2 00/10] building with clang

[Victor Toso]

Hey,

Just to keep this update! I can send a v3 if requested ;)

pushed upstream:
spice-protocol 01/10 macros: verify if __alloc_size__ works with clang
spice-protocol 02/10 macros: fix alignment issue reported by clang
spice-server 05/10 glz: WindowImageSegment lines lines_end as void*
spice-server 06/10 mjpeg and jpeg encoder: fix alignment warnings
spice-server 07/10 migration_protocol: use SPICE_MAGIC_CONST

need review:
spice-protocol 03/10 macros: new define to help with aligment warnings
spice-common 04/10 fix warnings about memory alignment
spice-server 08/10 unaligned type with spice_marshaller_reserve_space
spice-server 09/10 smartcard: use SPICE_ALIGNED_CAST
spice-server 10/10 fix warnings about memory alignment

Cheers,
 Victor Toso

On Fri, Aug 14, 2015 at 06:24:21PM +0200, Victor Toso wrote:
> Here we go with v2. Maybe I could squash some of these patches...
> Patch 0005 is acked-by Frediano;
> Thanks for any input,
> 
> changes from v1
> -> drop patch 02 and 12: on behaf of Frediano's better approach (upstream)
>    http://cgit.freedesktop.org/spice/spice-protocol/commit/?id=295d05e7330da45b705aa5081d140d7c84a931b0
> 
> -> Using SPICE_MAGIC_CONST on migration_protocol.h
> 
> -> patch 03 and 04: squashed (alignment for jpeg_encoder and mjpeg_encoder)
>    included Frediano's explanation on commit log and also a note that we
>    consider VM's data being aligned
> 
> -> patch 05 and 06: acked and pushed
> 
> -> patch 08: reserve_space is unaligned - using Frediano suggestions by
>    creating an unaligned type.
> 
> -> --enable-smartcard now + fixes
> 
> -> clang-noise patches are now using SPICE_ALIGNED_CAST and SPICE_UNALIGNED_CAST
>    to help track issues in the future.
> 
> Cheers,
>   Victor Toso
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel