[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