[Spice-devel] [PATCH spice-common] log: Use proper format attribute for spice_log

Frediano Ziglio fziglio at redhat.com
Wed Jan 2 13:02:26 UTC 2019


SPICE_ATTR_PRINTF uses __printf__ format attribute.
spice_log internally uses g_string_append_vprintf which uses
G_GNUC_PRINTF attribute.
G_GNUC_PRINTF can be __printf__ or gnu_printf format which in
some systems (currently Windows) can be different.
GLib 2.58 changed G_GNUC_PRINTF on Windows from using __printf__
attribute to gnu_printf.
To avoid problems in the future uses G_GNUC_PRINTF instead of
SPICE_ATTR_PRINTF for spice_log.

Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
---
 common/log.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/common/log.h b/common/log.h
index d352158..7c67e7a 100644
--- a/common/log.h
+++ b/common/log.h
@@ -37,7 +37,7 @@ void spice_log(GLogLevelFlags log_level,
                const char *strloc,
                const char *function,
                const char *format,
-               ...) SPICE_ATTR_PRINTF(4, 5);
+               ...) G_GNUC_PRINTF(4, 5);
 
 #define spice_return_if_fail(x) G_STMT_START {                          \
     if G_LIKELY(x) { } else {                                           \
-- 
2.20.1



More information about the Spice-devel mailing list