qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] w32: Always use standard instead of native form


From: Stefan Hajnoczi
Subject: Re: [Qemu-devel] [PATCH] w32: Always use standard instead of native format strings
Date: Sun, 23 Sep 2012 07:39:34 +0100
User-agent: Mutt/1.5.21 (2010-09-15)

On Wed, Aug 22, 2012 at 09:42:32PM +0200, Stefan Weil wrote:
> GLib 2.0 include files use __printf__ for the format attribute
> which resolves to native format strings on w32 hosts.
> 
> QEMU wants standard format strings instead of native format
> strings, so we simply change any declaration with __printf__
> to use __gnu_printf__.
> 
> This works because all basic printf functions support both
> kinds of format strings.
> 
> This fixes a compiler warning:
> 
> qapi/string-output-visitor.c: In function ‘print_type_int’:
> qapi/string-output-visitor.c:34:5: warning: unknown conversion type character 
> ‘l’ in format [-Wformat]
> qapi/string-output-visitor.c:34:5: warning: too many arguments for format 
> [-Wformat-extra-args]
> 
> Signed-off-by: Stefan Weil <address@hidden>
> ---
>  compiler.h |    5 +++++
>  1 file changed, 5 insertions(+)

Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches

Stefan



reply via email to

[Prev in Thread] Current Thread [Next in Thread]