Re: [PATCH v2] checkpatch: look for common misspellings

2014-09-12 Thread Masanari Iida
On Fri, Sep 12, 2014 at 1:45 PM, Joe Perches wrote: > On Fri, 2014-09-12 at 13:09 +0900, Masanari Iida wrote: >> Test with "reseting" case, codespell found 21, grep found 26. > > Hello Masanari. > > How did codespell find any uses of reseting? > What version of codespell are you using? > (I teste

Re: [PATCH v2] checkpatch: look for common misspellings

2014-09-11 Thread Joe Perches
On Fri, 2014-09-12 at 13:09 +0900, Masanari Iida wrote: > Test with "reseting" case, codespell found 21, grep found 26. Hello Masanari. How did codespell find any uses of reseting? What version of codespell are you using? (I tested with 1.7) Looking at the git tree for codespell, https://github

Re: [PATCH v2] checkpatch: look for common misspellings

2014-09-11 Thread Masanari Iida
Talking about codespell, it detected 76 "informations" in 3.17-rc4. " grep -R informations * |wc -l" found 120 typos. Test with "occured", codespell found 46, grep found 110. Test with "reseting" case, codespell found 21, grep found 26. So I expect about half of the incoming typos will be det

Re: [PATCH v2] checkpatch: look for common misspellings

2014-09-11 Thread Joe Perches
On Thu, 2014-09-11 at 09:19 +0200, Geert Uytterhoeven wrote: > On Thu, Sep 11, 2014 at 12:52 AM, Andrew Morton > wrote: > > On Mon, 8 Sep 2014 11:15:24 -0700 Kees Cook wrote: > >> Check for misspellings, based on Debian's lintian list. Several false > >> positives were removed, and several additi

Re: [PATCH v2] checkpatch: look for common misspellings

2014-09-11 Thread Kees Cook
On Thu, Sep 11, 2014 at 12:19 AM, Geert Uytterhoeven wrote: > On Thu, Sep 11, 2014 at 12:52 AM, Andrew Morton > wrote: >> On Mon, 8 Sep 2014 11:15:24 -0700 Kees Cook wrote: >> >>> Check for misspellings, based on Debian's lintian list. Several false >>> positives were removed, and several additi

Re: [PATCH v2] checkpatch: look for common misspellings

2014-09-11 Thread Geert Uytterhoeven
On Thu, Sep 11, 2014 at 12:52 AM, Andrew Morton wrote: > On Mon, 8 Sep 2014 11:15:24 -0700 Kees Cook wrote: > >> Check for misspellings, based on Debian's lintian list. Several false >> positives were removed, and several additional words added that were >> common in the kernel: >> >> backw

Re: [PATCH v2] checkpatch: look for common misspellings

2014-09-10 Thread Joe Perches
On Wed, 2014-09-10 at 15:52 -0700, Andrew Morton wrote: > Have a kernel joke: [] > @@ -553,6 +553,7 @@ jeffies||jiffies > +kubys|linus Gimmu Smftre/// -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo i

Re: [PATCH v2] checkpatch: look for common misspellings

2014-09-10 Thread Andrew Morton
On Mon, 8 Sep 2014 11:15:24 -0700 Kees Cook wrote: > Check for misspellings, based on Debian's lintian list. Several false > positives were removed, and several additional words added that were > common in the kernel: > > backword backwords > invalide valide > recieves >

Re: [PATCH v2] checkpatch: look for common misspellings

2014-09-10 Thread Joe Perches
On Wed, 2014-09-10 at 13:37 +0900, Masanari Iida wrote: > Hello Joe, Kees, Hello Masanari-san. > Sorry for late reply. > I was on holiday when the version 1 patch discussions were posted. No worries, holidays are far more important than patches like this... These patches are simple niceties, no

Re: [PATCH v2] checkpatch: look for common misspellings

2014-09-09 Thread Masanari Iida
Hello Joe, Kees, Sorry for late reply. I was on holiday when the version 1 patch discussions were posted. I am using codespell ( https://github.com/lucasdemarchi/codespell/ ). The codespell has its own typo dictionary. The dictionary format is typo->good (1 candidate) typo->good1,good2, (mult

Re: [PATCH v2] checkpatch: look for common misspellings

2014-09-08 Thread Joe Perches
On Mon, 2014-09-08 at 11:15 -0700, Kees Cook wrote: > Check for misspellings, based on Debian's lintian list. Several false > positives were removed, and several additional words added that were > common in the kernel: > > backword backwords > invalide valide > recieves > s

[PATCH v2] checkpatch: look for common misspellings

2014-09-08 Thread Kees Cook
Check for misspellings, based on Debian's lintian list. Several false positives were removed, and several additional words added that were common in the kernel: backword backwords invalide valide recieves singed unsinged While going back and fixing existing spellin