Messages in this thread | | | Date | Tue, 15 Sep 2015 16:01:23 +0100 | From | Andy Whitcroft <> | Subject | Re: possible new false positive in checkpatch |
| |
On Tue, Sep 15, 2015 at 05:50:40PM +0300, Tal Shorer wrote:
> > Yes it feels like that should be eliding them completely, and likely any > > following space as well, something like this: > > > > $s =~ s/$;+\s*//g; > > $c =~ s/$;+\s*//g; > > > Replacing the problematic lines with these fixes the issue. > > > Introduced in commit 9f5af480f4554aac12e002b6f5c2b04895857700: > > > checkpatch: improve SUSPECT_CODE_INDENT test > > > Commenting out these lines removes the warning. > > > > > > This pattern exists in many places around the kernel source. > > > Is this the intended behavior? > > > > Seems wrong to me. > > > > -apw > > Which git tree is checkpatch developed in? Linus's?
Yeah in Linus' tree.
-apw
| |