[Spice-commits] common/log.h

[GitLab Mirror]

 common/log.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 6dc745e877a12ed25634040dd6fa46fa5de694e2
Author: Frediano Ziglio <fziglio at redhat.com>
Date:   Wed Jan 2 13:02:06 2019 +0000

    log: Use proper format attribute for spice_log
    
    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>
    Acked-by: Christophe Fergeau <cfergeau at redhat.com>

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 {                                           \