diff mbox series

[v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined

Message ID 20201008165953.884599-1-marcandre.lureau@redhat.com
State New
Headers show
Series [v2] mingw: fix error __USE_MINGW_ANSI_STDIO redefined | expand

Commit Message

Marc-André Lureau Oct. 8, 2020, 4:59 p.m. UTC
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(-)

Comments

Peter Maydell Oct. 8, 2020, 5:09 p.m. UTC | #1
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
Stefan Weil Oct. 8, 2020, 5:25 p.m. UTC | #2
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
Philippe Mathieu-Daudé Oct. 8, 2020, 5:42 p.m. UTC | #3
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>
Laurent Vivier Oct. 12, 2020, 3:17 p.m. UTC | #4
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
diff mbox series

Patch

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"