Re: Coding style details (checkpatch)

Krzysztof Hałasa
2015-06-19T10:31:34Z

Frans Klaver <fransklaver@gmail.com> 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"?