On 16/1/26 13:58, Peter Maydell wrote:
This commit deals with various .c files that included system
headers that are already pulled in by osdep.h, where the .c
file includes osdep.h already itself.
This commit was created with scripts/clean-includes:
./scripts/clean-includes '--git' 'misc' 'hw/core' 'semihosting' 'target/arm'
'target/i386/kvm/kvm.c' 'target/loongarch' 'target/riscv' 'tools' 'util'
All .c should include qemu/osdep.h first. The script performs three
related cleanups:
* Ensure .c files include qemu/osdep.h first.
* Including it in a .h is redundant, since the .c already includes
it. Drop such inclusions.
* Likewise, including headers qemu/osdep.h includes is redundant.
Drop these, too.
Signed-off-by: Peter Maydell <[email protected]>
---
hw/core/machine-qmp-cmds.c | 1 -
semihosting/arm-compat-semi-stub.c | 1 -
target/arm/cpu32-stubs.c | 1 -
target/i386/kvm/kvm.c | 1 -
target/loongarch/csr.c | 1 -
target/riscv/csr.c | 1 -
tools/i386/qemu-vmsr-helper.c | 1 -
util/cpuinfo-aarch64.c | 1 -
8 files changed, 8 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
