Re: [PATCH 0/3] Add missing types to checkpatch.pl --list-types

From: Jean Delvare
Date: Sat Sep 02 2017 - 15:14:16 EST


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