Re: Coding style details (checkpatch)

From: Krzysztof HaÅasa
Date: Fri Jun 19 2015 - 06:31:37 EST


Frans Klaver <fransklaver@xxxxxxxxx> writes:

>> #define REG8_1(a0) ((const u16[8]){a0, a0 + 1, a0 + 2, a0 + 3})
>>
>> vs
>>
>> #define REG8_1(a0) ((const u16[8]) {a0, a0 + 1, a0 + 2, a0 + 3})
>> ^^^^^
>
> The prescribed style is to have no space between cast and castee. So,
> the top option.

Thanks. That's what I thought. It looks that checkpatch doesn't like
this:

ERROR: space required before the open brace '{'
+#define REG8_1(a0) ((const u16[8]){a0, a0 + 1, a0 + 2, a0 + 3})

Does this qualify as the "false positive"?
--
Krzysztof Halasa

Industrial Research Institute for Automation and Measurements PIAP
Al. Jerozolimskie 202, 02-486 Warsaw, Poland
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/