On 22 March 2018 at 17:57, <no-re...@patchew.org> wrote: > Checking PATCH 4/4: Remove unnecessary variables for function return value... > ERROR: return is not a function, parentheses are not required > #251: FILE: target/mips/dsp_helper.c:3281: > + return (temp[1] << 63) | (temp[0] >> 1);
This looks like a bug in checkpatch. I guess to fix it you'd need to make checkpatch count opening and closing parens in the line to see if it goes to 0 somewhere other than just before the ';'... thanks -- PMM
