linux-kernel - Re: [PATCH] checkpatch: fix typo and remove duplicate word

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CABJPP5D5TyJcnkmJfLU8-AE71SAuDOYKpLSoXvOy6889agUutA@mail.gmail.com>
Date:   Fri, 13 Nov 2020 19:18:37 +0530
From:   Dwaipayan Ray <dwaipayanray1@...il.com>
To:     Lukas Bulwahn <lukas.bulwahn@...il.com>
Cc:     Joe Perches <joe@...ches.com>,
        linux-kernel-mentees@...ts.linuxfoundation.org,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] checkpatch: fix typo and remove duplicate word

On Fri, Nov 13, 2020 at 7:04 PM Lukas Bulwahn <lukas.bulwahn@...il.com> wrote:
>
> On Fri, Nov 13, 2020 at 2:11 PM Dwaipayan Ray <dwaipayanray1@...il.com> wrote:
> >
> > Delete repeated word in scripts/checkpatch.pl.
> > "are are" -> "are"
> >
> > Fix typo "commments" -> "comments"
> >
> > Signed-off-by: Dwaipayan Ray <dwaipayanray1@...il.com>
>
> Does this make checkpatch.pl pass checkpatch without any further warnings?
>
> Nice idea to check if checkpatch.pl still fulfills the checkpatch
> rules we created...
>

Yes, all the rules pass.
$ ./scripts/checkpatch.pl --codespell --strict -f scripts/checkpatch.pl
gives:
total: 0 errors, 0 warnings, 10 checks, 7233 lines checked

9 of those checks are false positives. There is another misspelled
word:
#5003: FILE: scripts/checkpatch.pl:5003:
+## # falsly report the parameters of functions.

I will add that and resend.

Thanks,
Dwaipayan.