[PATCH spice-gtk v3 0/5] macOS fixes for spicy

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

 



From: Christophe de Dinechin <dinechin@xxxxxxxxxx>

This patch set mostly eliminates warnings detected by clang with
respect to type alignments. Vittorio Toso had submitted something
similar. In this version, I took into account comments by Christophe
Fergeau regarding how to know which casts were aligned and which ones
were not aligned. I added some instrumentation to detect the two
cases.

This requires an update to spice-common, see
https://lists.freedesktop.org/archives/spice-devel/2017-May/037505.html.
The reference to submodule  in this patch serie is updated with a
reference that can be fetched from https://github.com/c3d/spice-common.git.

Version 2 takes into account comments by Frediano Ziglio and Pavel Grunt,
specifically:

- Clarify purpose and behavior of macros in patch commit message
- Change one case to 'unaligned'
- Improve way to avoid 'unused variable' warning

Acked by: Christophe Fergeau <cfergeau@xxxxxxxxxx>

Version 3 takes into account comments made by Christophe Fergeau in
his ack message, specifically:

- Improve the log message description of SPICE_(UN)ALIGNED_CAST.
  Hopefully the new one is better.

- Make spaces after commas consistent

In addition, v3 re-enables ucontext on macOS, following comments
that this is the right thing to do for performance. This required the
elimination of one more alignment warning.

Christophe de Dinechin (5):
  Avoid clang warnings on casts with stricter alignment requirements
  Avoid warning about snprintf on non-Linux platforms
  Remove warning about unused variable when building on macOS
  Add check for macOS, disable ucontext on macOS (deprecated)
  Use SPICE_ALIGNED_CAST to silence warning with ucontext on macOS

 configure.ac                | 15 +++++++++++++++
 spice-common                |  2 +-
 src/channel-cursor.c        |  6 +++---
 src/channel-display-mjpeg.c |  2 +-
 src/continuation.h          |  6 ++++--
 src/decode-glz-tmpl.c       |  2 +-
 src/spice-channel.c         | 14 +++++++++-----
 src/spice-widget.c          |  4 ++--
 src/usbutil.c               |  2 +-
 9 files changed, 37 insertions(+), 16 deletions(-)

-- 
2.11.0 (Apple Git-81)

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://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]