[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/34] i386/hax: remove duplicated includes
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 07/34] i386/hax: remove duplicated includes |
Date: |
Fri, 22 Sep 2017 17:42:46 +0100 |
On 22 September 2017 at 16:39, Philippe Mathieu-Daudé <address@hidden> wrote:
> applied using ./scripts/clean-includes
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> include/sysemu/hax.h | 1 -
> target/i386/hax-darwin.h | 3 ---
> target/i386/hax-windows.h | 2 --
> target/i386/hax-darwin.c | 8 ++------
> 4 files changed, 2 insertions(+), 12 deletions(-)
I suspect the includes in hax-windows.h could be trimmed a bit
more (since we're guaranteed that if CONFIG_WIN32 then we've
included sysemu/os-win32.h), but this is fine as it stands.
Reviewed-by: Peter Maydell <address@hidden>
thanks
-- PMM
- [Qemu-devel] [PATCH 03/34] block: remove "qemu/osdep.h" from header file, (continued)
- [Qemu-devel] [PATCH 03/34] block: remove "qemu/osdep.h" from header file, Philippe Mathieu-Daudé, 2017/09/22
- [Qemu-devel] [PATCH 04/34] misc: remove headers implicitly included, Philippe Mathieu-Daudé, 2017/09/22
- [Qemu-devel] [PATCH 05/34] misc: remove duplicated includes, Philippe Mathieu-Daudé, 2017/09/22
- [Qemu-devel] [PATCH 06/34] ppc: remove duplicated includes, Philippe Mathieu-Daudé, 2017/09/22
- [Qemu-devel] [PATCH 07/34] i386/hax: remove duplicated includes, Philippe Mathieu-Daudé, 2017/09/22
- Re: [Qemu-devel] [PATCH 07/34] i386/hax: remove duplicated includes,
Peter Maydell <=
- [Qemu-devel] [PATCH 08/34] nios2: remove duplicated includes, Philippe Mathieu-Daudé, 2017/09/22
- [Qemu-devel] [PATCH 09/34] misc: avoid "include/" in include path, Philippe Mathieu-Daudé, 2017/09/22
- [Qemu-devel] [PATCH 11/34] i386/pc: use TYPE_PORT92, Philippe Mathieu-Daudé, 2017/09/22
- [Qemu-devel] [PATCH 10/34] amd_iommu: avoid needless includes in header file, Philippe Mathieu-Daudé, 2017/09/22
- [Qemu-devel] [PATCH 12/34] misc: remove old i386 dependency, Philippe Mathieu-Daudé, 2017/09/22