[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 01/19] Use #include "..." for our own headers
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 01/19] Use #include "..." for our own headers, <...> for others |
Date: |
Thu, 01 Feb 2018 07:57:19 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Thomas Huth <address@hidden> writes:
> On 31.01.2018 15:48, Markus Armbruster wrote:
>> System headers should be included with <...>, our own headers with
>> "...". Offenders tracked down with an ugly, brittle and probably
>> buggy Perl script. Previous iteration was commit a9c94277f0.
>>
>> Put the cleaned up system header includes first, except for the ones
>> the next commit will delete.
>
> That's a little bit of code churn ... why not delete them here
> immediately, or simply ignore these headers here and just delete them in
> the next patch?
Ignore won't do, as scripts/clean-includes won't find them then.
Delete is possible, but requires still more explanation in the commit
message. Worthwhile?
>> While there, separate #include from file comment with a blank line,
>>
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
> [...]
>> diff --git a/target/s390x/gen-features.c b/target/s390x/gen-features.c
>> index 0570f597ec..9968974fbe 100644
>> --- a/target/s390x/gen-features.c
>> +++ b/target/s390x/gen-features.c
>> @@ -13,8 +13,8 @@
>> */
>>
>>
>
> While you're at it, remove one of the two blank lines?
Yes.
>> -#include "inttypes.h"
>> -#include "stdio.h"
>> +#include <inttypes.h>
>> +#include <stdio.h>
>> #include "cpu_features_def.h"
>
> Above remarks are just nits, anyway patch looks fine, so:
>
> Reviewed-by: Thomas Huth <address@hidden>
Thanks!
- Re: [Qemu-devel] [PATCH v2 01/19] Use #include "..." for our own headers, <...> for others,
Markus Armbruster <=