linux-kernel - Re: [PATCH 17/17] scripts/spelling.txt: Add descriptors correction

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] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 10 Jun 2020 10:09:36 +0100
From:   Kieran Bingham <kieran.bingham+renesas@...asonboard.com>
To:     Joe Perches <joe@...ches.com>,
        Kieran Bingham <kieran.bingham@...asonboard.com>
Cc:     linux-renesas-soc@...r.kernel.org,
        Andrew Morton <akpm@...ux-foundation.org>,
        Colin Ian King <colin.king@...onical.com>,
        Geert Uytterhoeven <geert+renesas@...der.be>,
        Xiong <xndchn@...il.com>,
        Chris Paterson <chris.paterson2@...esas.com>,
        Paul Walmsley <paul.walmsley@...ive.com>,
        Luca Ceresoli <luca@...aceresoli.net>,
        Jonathan Neuschäfer <j.neuschaefer@....net>,
        open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 17/17] scripts/spelling.txt: Add descriptors correction

Hi Joe,

On 09/06/2020 17:08, Joe Perches wrote:
> On Tue, 2020-06-09 at 13:46 +0100, Kieran Bingham wrote:
>> After coming across the s/decriptors/descriptors/ spelling error twice,
>> a scan of the tree showed it was a pervasive mistake.
> []
>> diff --git a/scripts/spelling.txt b/scripts/spelling.txt
> []
>> @@ -404,6 +404,7 @@ decendants||descendants
>>  decompres||decompress
>>  decsribed||described
>>  decription||description
>> +decriptors||descriptors
> 
> spelling.txt does not check singular and plural
> uses unless each is described separately so please
> add the singular one too:
> 
> decriptor|descriptor
> 
> $ git grep -i -w decriptor | wc -l
> 18
> $ git grep -i -w decriptors | wc -l
> 12

I believe my conversion was based on the non-pluralised version, so I
think I changed all occurences, but then foolishly somehow ended up
using the plural for the commit message, which translated into being the
entry I put in spelling.txt. Sorry ... :-S

I'll fix up, and re-validate any missing fixups after the rebase. I
think I originally did this at about v5.0, and I already picked up a
couple extra when I did the rebase, and now you've highlighted the
plural - I bet I missed some non-plural versions too ;-)

Will re-check and correct.

Thanks.

--
Kieran