[Top][All Lists]
[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
- Re: [Qemu-devel] [PATCH] w32: Always use standard instead of native format strings,
Stefan Hajnoczi <=