lkml.org 
[lkml]   [2017]   [Sep]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 0/3] Add missing types to checkpatch.pl --list-types
Hi Joe,

On Sat, 02 Sep 2017 09:25:29 -0700, Joe Perches wrote:
> On Sat, 2017-09-02 at 17:51 +0200, Jean Delvare wrote:
> > [PATCH 1/3] checkpatch: fix typo in comment
> > [PATCH 2/3] checkpatch: rename variables to avoid confusion
> > [PATCH 3/3] checkpatch: add 6 missing types to --list-types
>
> Andrew, can you please pick these up? Thanks.
>
> Jean, that's a pretty obscure defect.
> Thanks for handling it.

You're welcome.

> btw: how or why did you find it?

I wrote a patch which introduces a lot of long lines, and I am fine
with that in this specific case, so I wanted to ask checkpatch to print
everything BUT line-over-80-columns warnings. I used --list-types to
find out the internal name for it... and couldn't see anything matching
in the list. But I knew it had to exist, so looked into the code.

--
Jean Delvare
SUSE L3 Support

\
 
 \ /
  Last update: 2017-09-02 21:14    [W:0.054 / U:5.088 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site