* [PATCH v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined
@ 2020-10-08 16:59 marcandre.lureau
2020-10-08 17:09 ` Peter Maydell
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: marcandre.lureau @ 2020-10-08 16:59 UTC (permalink / raw)
To: qemu-devel; +Cc: sw, armbru, Marc-André Lureau
From: Marc-André Lureau <marcandre.lureau@redhat.com>
Always put osdep.h first, and remove redundant stdlib.h include.
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
---
migration/dirtyrate.c | 3 ++-
tests/test-bitmap.c | 1 -
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/migration/dirtyrate.c b/migration/dirtyrate.c
index 68577ef250..47f761e67a 100644
--- a/migration/dirtyrate.c
+++ b/migration/dirtyrate.c
@@ -10,8 +10,9 @@
* See the COPYING file in the top-level directory.
*/
-#include <zlib.h>
#include "qemu/osdep.h"
+
+#include <zlib.h>
#include "qapi/error.h"
#include "cpu.h"
#include "qemu/config-file.h"
diff --git a/tests/test-bitmap.c b/tests/test-bitmap.c
index 2f5b71458a..8db4f67883 100644
--- a/tests/test-bitmap.c
+++ b/tests/test-bitmap.c
@@ -8,7 +8,6 @@
* Author: Peter Xu <peterx@redhat.com>
*/
-#include <stdlib.h>
#include "qemu/osdep.h"
#include "qemu/bitmap.h"
--
2.28.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined
2020-10-08 16:59 [PATCH v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined marcandre.lureau
@ 2020-10-08 17:09 ` Peter Maydell
2020-10-08 17:25 ` Stefan Weil
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Peter Maydell @ 2020-10-08 17:09 UTC (permalink / raw)
To: Marc-André Lureau; +Cc: Stefan Weil, QEMU Developers, Markus Armbruster
On Thu, 8 Oct 2020 at 18:03, <marcandre.lureau@redhat.com> wrote:
>
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
>
> Always put osdep.h first, and remove redundant stdlib.h include.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
> migration/dirtyrate.c | 3 ++-
> tests/test-bitmap.c | 1 -
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
thanks
-- PMM
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined
2020-10-08 16:59 [PATCH v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined marcandre.lureau
2020-10-08 17:09 ` Peter Maydell
@ 2020-10-08 17:25 ` Stefan Weil
2020-10-08 17:42 ` Philippe Mathieu-Daudé
2020-10-12 15:17 ` Laurent Vivier
3 siblings, 0 replies; 5+ messages in thread
From: Stefan Weil @ 2020-10-08 17:25 UTC (permalink / raw)
To: marcandre.lureau, qemu-devel; +Cc: armbru
Am 08.10.20 um 18:59 schrieb marcandre.lureau@redhat.com:
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
>
> Always put osdep.h first, and remove redundant stdlib.h include.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
> migration/dirtyrate.c | 3 ++-
> tests/test-bitmap.c | 1 -
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/migration/dirtyrate.c b/migration/dirtyrate.c
> index 68577ef250..47f761e67a 100644
> --- a/migration/dirtyrate.c
> +++ b/migration/dirtyrate.c
> @@ -10,8 +10,9 @@
> * See the COPYING file in the top-level directory.
> */
>
> -#include <zlib.h>
> #include "qemu/osdep.h"
> +
> +#include <zlib.h>
> #include "qapi/error.h"
> #include "cpu.h"
> #include "qemu/config-file.h"
> diff --git a/tests/test-bitmap.c b/tests/test-bitmap.c
> index 2f5b71458a..8db4f67883 100644
> --- a/tests/test-bitmap.c
> +++ b/tests/test-bitmap.c
> @@ -8,7 +8,6 @@
> * Author: Peter Xu <peterx@redhat.com>
> */
>
> -#include <stdlib.h>
> #include "qemu/osdep.h"
> #include "qemu/bitmap.h"
>
Reviewed-by: Stefan Weil <sw@weilnetz.de>
Thank you,
Stefan
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined
2020-10-08 16:59 [PATCH v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined marcandre.lureau
2020-10-08 17:09 ` Peter Maydell
2020-10-08 17:25 ` Stefan Weil
@ 2020-10-08 17:42 ` Philippe Mathieu-Daudé
2020-10-12 15:17 ` Laurent Vivier
3 siblings, 0 replies; 5+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-10-08 17:42 UTC (permalink / raw)
To: marcandre.lureau, qemu-devel; +Cc: sw, armbru
On 10/8/20 6:59 PM, marcandre.lureau@redhat.com wrote:
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
>
> Always put osdep.h first, and remove redundant stdlib.h include.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
> migration/dirtyrate.c | 3 ++-
> tests/test-bitmap.c | 1 -
> 2 files changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined
2020-10-08 16:59 [PATCH v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined marcandre.lureau
` (2 preceding siblings ...)
2020-10-08 17:42 ` Philippe Mathieu-Daudé
@ 2020-10-12 15:17 ` Laurent Vivier
3 siblings, 0 replies; 5+ messages in thread
From: Laurent Vivier @ 2020-10-12 15:17 UTC (permalink / raw)
To: marcandre.lureau, qemu-devel; +Cc: qemu-trivial@nongnu.org, sw, armbru
Le 08/10/2020 à 18:59, marcandre.lureau@redhat.com a écrit :
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
>
> Always put osdep.h first, and remove redundant stdlib.h include.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
> migration/dirtyrate.c | 3 ++-
> tests/test-bitmap.c | 1 -
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/migration/dirtyrate.c b/migration/dirtyrate.c
> index 68577ef250..47f761e67a 100644
> --- a/migration/dirtyrate.c
> +++ b/migration/dirtyrate.c
> @@ -10,8 +10,9 @@
> * See the COPYING file in the top-level directory.
> */
>
> -#include <zlib.h>
> #include "qemu/osdep.h"
> +
> +#include <zlib.h>
> #include "qapi/error.h"
> #include "cpu.h"
> #include "qemu/config-file.h"
> diff --git a/tests/test-bitmap.c b/tests/test-bitmap.c
> index 2f5b71458a..8db4f67883 100644
> --- a/tests/test-bitmap.c
> +++ b/tests/test-bitmap.c
> @@ -8,7 +8,6 @@
> * Author: Peter Xu <peterx@redhat.com>
> */
>
> -#include <stdlib.h>
> #include "qemu/osdep.h"
> #include "qemu/bitmap.h"
>
>
Applied to my trivial-patches branch.
Thanks,
Laurent
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-10-12 15:18 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-10-08 16:59 [PATCH v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined marcandre.lureau
2020-10-08 17:09 ` Peter Maydell
2020-10-08 17:25 ` Stefan Weil
2020-10-08 17:42 ` Philippe Mathieu-Daudé
2020-10-12 15:17 ` Laurent Vivier
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).