Messages in this thread | | | From | (Krzysztof Hałasa) | Subject | Re: Coding style details (checkpatch) | Date | Fri, 19 Jun 2015 12:31:24 +0200 |
| |
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"? -- Krzysztof Halasa
Industrial Research Institute for Automation and Measurements PIAP Al. Jerozolimskie 202, 02-486 Warsaw, Poland
| |