2015-06-26 04:14:53

by Maninder Singh

[permalink] [raw]
Subject: [PATCH 1/1] scripts/spelling.txt: Adding misspelled word for check.

misspelled words for check:-
chcek
chck
cehck

I myself did these spell mistakes in changelog for patches,
Thus suggesting to add in spelling.txt, so that checkpatch.pl
warns it earlier.
References:-

./arch/powerpc/kernel/exceptions-64e.S:456: . . . make sure you chcek
https://lkml.org/lkml/2015/6/25/289
./arch/x86/mm/pageattr.c:1368: * No need to cehck in that case

Signed-off-by: Maninder Singh <[email protected]>
---
scripts/spelling.txt | 3 +++
1 file changed, 3 insertions(+)

diff --git a/scripts/spelling.txt b/scripts/spelling.txt
index bb8e4d0..958d054 100644
--- a/scripts/spelling.txt
+++ b/scripts/spelling.txt
@@ -187,6 +187,7 @@ capatibilities||capabilities
carefuly||carefully
cariage||carriage
catagory||category
+cehck||check
challange||challenge
challanges||challenges
chanell||channel
@@ -199,6 +200,8 @@ charactor||character
charater||character
charaters||characters
charcter||character
+chcek||check
+chck||check
checksuming||checksumming
childern||children
childs||children
--
1.7.9.5


2015-06-26 17:37:31

by Kees Cook

[permalink] [raw]
Subject: Re: [PATCH 1/1] scripts/spelling.txt: Adding misspelled word for check.

On Thu, Jun 25, 2015 at 9:13 PM, Maninder Singh <[email protected]> wrote:
> misspelled words for check:-
> chcek
> chck
> cehck
>
> I myself did these spell mistakes in changelog for patches,
> Thus suggesting to add in spelling.txt, so that checkpatch.pl
> warns it earlier.
> References:-
>
> ./arch/powerpc/kernel/exceptions-64e.S:456: . . . make sure you chcek
> https://lkml.org/lkml/2015/6/25/289
> ./arch/x86/mm/pageattr.c:1368: * No need to cehck in that case
>
> Signed-off-by: Maninder Singh <[email protected]>

Thanks! These aren't very common, but there aren't any false positives. :)

Acked-by: Kees Cook <[email protected]>

-Kees

> ---
> scripts/spelling.txt | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/scripts/spelling.txt b/scripts/spelling.txt
> index bb8e4d0..958d054 100644
> --- a/scripts/spelling.txt
> +++ b/scripts/spelling.txt
> @@ -187,6 +187,7 @@ capatibilities||capabilities
> carefuly||carefully
> cariage||carriage
> catagory||category
> +cehck||check
> challange||challenge
> challanges||challenges
> chanell||channel
> @@ -199,6 +200,8 @@ charactor||character
> charater||character
> charaters||characters
> charcter||character
> +chcek||check
> +chck||check
> checksuming||checksumming
> childern||children
> childs||children
> --
> 1.7.9.5
>



--
Kees Cook
Chrome OS Security

2015-07-09 22:38:58

by Andrew Morton

[permalink] [raw]
Subject: Re: [PATCH 1/1] scripts/spelling.txt: Adding misspelled word for check.

On Fri, 26 Jun 2015 10:37:26 -0700 Kees Cook <[email protected]> wrote:

> On Thu, Jun 25, 2015 at 9:13 PM, Maninder Singh <[email protected]> wrote:
> > misspelled words for check:-
> > chcek
> > chck
> > cehck
> >
> > I myself did these spell mistakes in changelog for patches,
> > Thus suggesting to add in spelling.txt, so that checkpatch.pl
> > warns it earlier.
> > References:-
> >
> > ./arch/powerpc/kernel/exceptions-64e.S:456: . . . make sure you chcek
> > https://lkml.org/lkml/2015/6/25/289
> > ./arch/x86/mm/pageattr.c:1368: * No need to cehck in that case
> >
> > Signed-off-by: Maninder Singh <[email protected]>
>
> Thanks! These aren't very common, but there aren't any false positives. :)
>
> Acked-by: Kees Cook <[email protected]>

Well I'm going to sneak this in there to.


From: Andrew Morton <[email protected]>
Subject: scripts-spellingtxt-adding-misspelled-word-for-check-fix

add whcih->which, whcih I always get wrong

Cc: Kees Cook <[email protected]>
Cc: Maninder Singh <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
---

scripts/spelling.txt | 1 +
1 file changed, 1 insertion(+)

diff -puN scripts/spelling.txt~scripts-spellingtxt-adding-misspelled-word-for-check-fix scripts/spelling.txt
--- a/scripts/spelling.txt~scripts-spellingtxt-adding-misspelled-word-for-check-fix
+++ a/scripts/spelling.txt
@@ -1031,6 +1031,7 @@ visiters||visitors
vitual||virtual
wating||waiting
whataver||whatever
+whcih||which
whenver||whenever
wheter||whether
whe||when
_

2015-07-09 22:47:21

by Kees Cook

[permalink] [raw]
Subject: Re: [PATCH 1/1] scripts/spelling.txt: Adding misspelled word for check.

On Thu, Jul 9, 2015 at 3:38 PM, Andrew Morton <[email protected]> wrote:
> On Fri, 26 Jun 2015 10:37:26 -0700 Kees Cook <[email protected]> wrote:
>
>> On Thu, Jun 25, 2015 at 9:13 PM, Maninder Singh <[email protected]> wrote:
>> > misspelled words for check:-
>> > chcek
>> > chck
>> > cehck
>> >
>> > I myself did these spell mistakes in changelog for patches,
>> > Thus suggesting to add in spelling.txt, so that checkpatch.pl
>> > warns it earlier.
>> > References:-
>> >
>> > ./arch/powerpc/kernel/exceptions-64e.S:456: . . . make sure you chcek
>> > https://lkml.org/lkml/2015/6/25/289
>> > ./arch/x86/mm/pageattr.c:1368: * No need to cehck in that case
>> >
>> > Signed-off-by: Maninder Singh <[email protected]>
>>
>> Thanks! These aren't very common, but there aren't any false positives. :)
>>
>> Acked-by: Kees Cook <[email protected]>
>
> Well I'm going to sneak this in there to.
>
>
> From: Andrew Morton <[email protected]>
> Subject: scripts-spellingtxt-adding-misspelled-word-for-check-fix
>
> add whcih->which, whcih I always get wrong

Hehe. :) Perfect!

Acked-by: Kees Cook <[email protected]>

-Kees

>
> Cc: Kees Cook <[email protected]>
> Cc: Maninder Singh <[email protected]>
> Signed-off-by: Andrew Morton <[email protected]>
> ---
>
> scripts/spelling.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff -puN scripts/spelling.txt~scripts-spellingtxt-adding-misspelled-word-for-check-fix scripts/spelling.txt
> --- a/scripts/spelling.txt~scripts-spellingtxt-adding-misspelled-word-for-check-fix
> +++ a/scripts/spelling.txt
> @@ -1031,6 +1031,7 @@ visiters||visitors
> vitual||virtual
> wating||waiting
> whataver||whatever
> +whcih||which
> whenver||whenever
> wheter||whether
> whe||when
> _
>



--
Kees Cook
Chrome OS Security