[Spice-devel] [PATCH v3 0/2] macOS enablement patches

Christophe de Dinechin christophe at dinechin.org
Thu May 18 13:24:51 UTC 2017


From: Christophe de Dinechin <dinechin at redhat.com>

This patch set 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.

v2 changes:

- Fix alignment used for checking (alignment of pointed type, not pointer)

- Disable checking entirely if NDEBUG is defined.

- Add G_UNLIKELY to indicate that the branch is not likely to be taken

I saw no obvious equivalent of NDEBUG in Spice code, so I used NDEBUG,
which is the macro used to disable assert() defined in <assert.h>.

v3 changes:

- Group all stylistic changes together

- Additional style consistency cleanup suggested by Frediano

- Disable misaligned check unless compiled with CFLAGS=-DSPICE_DEBUG_ALIGNMENT

Christophe de Dinechin (2):
  Style adjustment - Making code match surrounding style
  Avoid clang warnings on casts with stricter alignment requirements

 common/canvas_base.c |  14 ++++--
 common/mem.c         |  23 +++++++++
 common/mem.h         |  43 +++++++++++++++-
 common/sw_canvas.c   | 138 +++++++++++++++++++++++++--------------------------
 4 files changed, 140 insertions(+), 78 deletions(-)

-- 
2.11.0 (Apple Git-81)



More information about the Spice-devel mailing list