[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] block: Clean up includes
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH] block: Clean up includes |
Date: |
Tue, 19 Jan 2016 15:50:03 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 18.01.2016 um 19:01 hat Peter Maydell geschrieben:
> Clean up includes so that osdep.h is included first and headers
> which it implies are not included manually.
>
> This commit was created with scripts/clean-includes.
>
> Signed-off-by: Peter Maydell <address@hidden>
> --- a/block/archipelago.c
> +++ b/block/archipelago.c
> @@ -50,6 +50,7 @@
> *
> */
>
> +#include "qemu/osdep.h"
> #include "qemu-common.h"
> #include "block/block_int.h"
> #include "qemu/error-report.h"
qemu/osdep.h is already included by qemu-common.h. Do you intend to
remove all qemu/osdep.h includes from all header files (and possibly
error out if it hasn't been included before) as soon as most/all .c
files include it explicitly?
This is what git grep says on include/:
include/exec/cpu-defs.h:#include "qemu/osdep.h"
include/exec/helper-head.h:#include "qemu/osdep.h"
include/fpu/softfloat.h:#include "qemu/osdep.h"
include/qemu-common.h:#include "qemu/osdep.h"
include/qemu/bitmap.h:#include "qemu/osdep.h"
include/qemu/module.h:#include "qemu/osdep.h"
include/sysemu/seccomp.h:#include "qemu/osdep.h"
Kevin