[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v4 5/7] tests: New make target check-source
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [RFC v4 5/7] tests: New make target check-source |
Date: |
Mon, 27 May 2019 07:10:31 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> On 23/05/19 21:57, Markus Armbruster wrote:
>>>> A large number of headers don't pass this test, by design or by
>>>> accident. To keep things more manageable, exclude all headers outside
>>>> include/ for now.
>>> A lot of these, either in include/ or outside, are _meant_ to be
>>> included many times. What about renaming those to .inc.h and
>>> whitelisting them in the script?
>> Yes, that would be nice.
>
> Another suggestion: are there headers that cannot even be included once
> (due to dependencies)? Is it worth including a test for those even in
> the first iteration?
>
> .inc.h files would skip that test too.
I'm not sure I get what you mean.
Most headers failing the test fail it in the first #include: they fail
to conform to 2. Headers should normally include everything they need
beyond osdep.h.
The only way to fail in the second #include is a missing header guard.
If it's missing intentionally, it's "_meant_ to be included many times",
and you propose renaming to .inc.h. Else, easy fix.
I think I'll make a list of headers that fail in the second #include,
and try to sort them into "intentional" and "bug" buckets.
- Re: [Qemu-devel] [RFC v4 1/7] Makefile: Remove code to smooth transition to config.status, (continued)
Re: [Qemu-devel] [RFC v4 5/7] tests: New make target check-source, Philippe Mathieu-Daudé, 2019/05/24
[Qemu-devel] [RFC v4 6/7] tests: Don't limit check-headers to include/, Markus Armbruster, 2019/05/23