For blizzard, pl110 and tc6393xb this is harmless, but for pxa2xx
Coverity noticed that it is used inside an "if" statement.
Fix it because it's the file with the highest number of defects
in the whole QEMU tree!
Signed-off-by: Paolo Bonzini <address@hidden>
---
hw/display/blizzard_template.h | 10 +++++-----
hw/display/pl110_template.h | 6 +++---
hw/display/pxa2xx_template.h | 8 ++++----
hw/display/tc6393xb_template.h | 8 ++++----
4 files changed, 22 insertions(+), 23 deletions(-)
diff --git a/hw/display/blizzard_template.h b/hw/display/blizzard_template.h
index a8a8899..b899a29 100644
--- a/hw/display/blizzard_template.h
+++ b/hw/display/blizzard_template.h
@@ -21,21 +21,21 @@
#define SKIP_PIXEL(to) to += deststep
#if DEPTH == 8
# define PIXEL_TYPE uint8_t
-# define COPY_PIXEL(to, from) *to = from; SKIP_PIXEL(to)
+# define COPY_PIXEL(to, from) *to = from, SKIP_PIXEL(to)