[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/8] scripts/clean-includes: Enhance to handle h
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 4/8] scripts/clean-includes: Enhance to handle header files |
Date: |
Thu, 18 Feb 2016 18:36:10 +0000 |
On 18 February 2016 at 18:05, Peter Maydell <address@hidden> wrote:
> Enhance clean-includes to handle header files as well as .c source
> files. For headers we merely remove all the redundant #include
> lines, including any includes of qemu/osdep.h itself.
>
> There is a simple mollyguard on the include file processing to
> skip a few key headers like osdep.h itself, to avoid producing
> bad patches if the script is run on every file in include/.
>
> Signed-off-by: Peter Maydell <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
Hmm, I'm not sure why this ended up with an R-b tag from Eric
because I'm pretty sure this is the first time I've sent it to
the list. Sorry about that, and please ignore that r-b tag...
thanks
-- PMM
- Re: [Qemu-devel] [PATCH 7/8] all: Clean up includes, (continued)
- [Qemu-devel] [PATCH 8/8] include: Clean up includes, Peter Maydell, 2016/02/18
- [Qemu-devel] [PATCH 2/8] osdep.h: Define macros for the benefit of C++ before C++11, Peter Maydell, 2016/02/18
- [Qemu-devel] [PATCH 6/8] osdep.h: Include config-target.h if NEED_CPU_H is defined, Peter Maydell, 2016/02/18
- [Qemu-devel] [PATCH 3/8] disas/arm-a64.cc: Include osdep.h first, Peter Maydell, 2016/02/18
- [Qemu-devel] [PATCH 4/8] scripts/clean-includes: Enhance to handle header files, Peter Maydell, 2016/02/18
- [Qemu-devel] [PATCH 5/8] scripts/clean-includes: Add --all option, Peter Maydell, 2016/02/18
- [Qemu-devel] [PATCH 1/8] cpu: Clean up includes, Peter Maydell, 2016/02/18
- Re: [Qemu-devel] [PATCH 0/8] more include cleaning, Eric Blake, 2016/02/18