[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Clean up includes
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] Clean up includes |
Date: |
Wed, 13 Mar 2019 18:36:49 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
BALATON Zoltan <address@hidden> writes:
> On Wed, 13 Mar 2019, Markus Armbruster wrote:
>> 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, with the changes
>> to the following files manually reverted:
>>
>> contrib/libvhost-user/libvhost-user-glib.h
>> contrib/libvhost-user/libvhost-user.c
>> contrib/libvhost-user/libvhost-user.h
>> linux-user/mips64/cpu_loop.c
>> linux-user/mips64/signal.c
>> linux-user/sparc64/cpu_loop.c
>> linux-user/sparc64/signal.c
>> linux-user/x86_64/cpu_loop.c
>> linux-user/x86_64/signal.c
>> slirp/src/*
>> target/s390x/gen-features.c
>> tests/migration/s390x/a-b-bios.c
>> tests/test-rcu-simpleq.c
>> tests/test-rcu-tailq.c
>> tests/uefi-test-tools/UefiTestToolsPkg/BiosTablesTest/BiosTablesTest.c
>>
>> We're in the process of spinning out slirp/. tests/uefi-test-tools/
>> is guest software. The remaining reverts are the same as in commit
>> b7d89466dde.
>>
>> Signed-off-by: Markus Armbruster <address@hidden>
[...]
>> diff --git a/hw/display/ati_2d.c b/hw/display/ati_2d.c
>> index bc98ba6eeb..f31b3c27c7 100644
>> --- a/hw/display/ati_2d.c
>> +++ b/hw/display/ati_2d.c
>> @@ -7,6 +7,7 @@
>> * This work is licensed under the GNU GPL license version 2 or later.
>> */
>>
>> +#include "qemu/osdep.h"
>> #include "ati_int.h"
>> #include "ati_regs.h"
>> #include "qemu/log.h"
>> diff --git a/hw/display/ati_dbg.c b/hw/display/ati_dbg.c
>> index 1e6c32624e..b045f81d06 100644
>> --- a/hw/display/ati_dbg.c
>> +++ b/hw/display/ati_dbg.c
>> @@ -1,3 +1,4 @@
>> +#include "qemu/osdep.h"
>> #include "ati_int.h"
>>
>> #ifdef DEBUG_ATI
>> diff --git a/hw/display/ati_int.h b/hw/display/ati_int.h
>> index a6f3e20e63..2f426064cf 100644
>> --- a/hw/display/ati_int.h
>> +++ b/hw/display/ati_int.h
>> @@ -9,7 +9,6 @@
>> #ifndef ATI_INT_H
>> #define ATI_INT_H
>>
>> -#include "qemu/osdep.h"
>
> What's wrong with ati_int.h including osdep.h first and everything
> else including ati_int.h first? I think it was OK that way so unless
> there's a good reason to explicitely include osdep in all files that
> also include ati_int.h I think these should not be changed. For the
> ati model we need ati_int.h included first so it's OK to include
> osdep.h from there.
./HACKING explains:
1.2. Include directives
Order include directives as follows:
#include "qemu/osdep.h" /* Always first... */
#include <...> /* then system headers... */
#include "..." /* and finally QEMU headers. */
The "qemu/osdep.h" header contains preprocessor macros that affect the
behavior
of core system headers like <stdint.h>. It must be the first include so
that
core system headers included by external libraries get the preprocessor
macros
that QEMU depends on.
Do not include "qemu/osdep.h" from header files since the .c file will have
already included it.