[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH v3 4/4] osdep: Fix mingw compilat
From: |
Thomas Huth |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH v3 4/4] osdep: Fix mingw compilation regarding stdio formats |
Date: |
Fri, 3 May 2019 07:11:43 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 |
On 03/05/2019 02.37, Cao Jiaxi wrote:
> I encountered the following compilation error on mingw:
>
> /mnt/d/qemu/include/qemu/osdep.h:97:9: error: '__USE_MINGW_ANSI_STDIO' macro
> redefined [-Werror,-Wmacro-redefined]
> #define __USE_MINGW_ANSI_STDIO 1
> ^
> /mnt/d/llvm-mingw/aarch64-w64-mingw32/include/_mingw.h:433:9: note: previous
> definition is here
> #define __USE_MINGW_ANSI_STDIO 0 /* was not defined so it should be 0 */
>
> It turns out that __USE_MINGW_ANSI_STDIO must be set before any
> system headers are included, not just before stdio.h.
>
> Signed-off-by: Cao Jiaxi <address@hidden>
> ---
> include/qemu/osdep.h | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/include/qemu/osdep.h b/include/qemu/osdep.h
> index 303d315c5d..af2b91f0b8 100644
> --- a/include/qemu/osdep.h
> +++ b/include/qemu/osdep.h
> @@ -85,17 +85,17 @@ extern int daemon(int, int);
> #endif
> #endif
>
> +/* enable C99/POSIX format strings (needs mingw32-runtime 3.15 or later) */
> +#ifdef __MINGW32__
> +#define __USE_MINGW_ANSI_STDIO 1
> +#endif
> +
> #include <stdarg.h>
> #include <stddef.h>
> #include <stdbool.h>
> #include <stdint.h>
> #include <sys/types.h>
> #include <stdlib.h>
> -
> -/* enable C99/POSIX format strings (needs mingw32-runtime 3.15 or later) */
> -#ifdef __MINGW32__
> -#define __USE_MINGW_ANSI_STDIO 1
> -#endif
> #include <stdio.h>
>
> #include <string.h>
>
Reviewed-by: Thomas Huth <address@hidden>
- Re: [Qemu-trivial] [Qemu-devel] [PATCH v3 4/4] osdep: Fix mingw compilation regarding stdio formats,
Thomas Huth <=