Messages in this thread | | | Date | Thu, 11 Sep 2014 07:12:24 -0700 | Subject | Re: [PATCH v2] checkpatch: look for common misspellings | From | Kees Cook <> |
| |
On Thu, Sep 11, 2014 at 12:19 AM, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > On Thu, Sep 11, 2014 at 12:52 AM, Andrew Morton > <akpm@linux-foundation.org> wrote: >> On Mon, 8 Sep 2014 11:15:24 -0700 Kees Cook <keescook@chromium.org> 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 >>> singed unsinged >>> >>> While going back and fixing existing spelling mistakes isn't a high >>> priority, it'd be nice to try to catch them before they hit the tree. >> >> I have a feeling this is going to be a rat hole and that >> scripts/spelling.txt will grow to consume the planet. Oh well, whatev. > > What about making checkpatch use the codespell dictionay if codespell > is installed? > > Codespell is in Ubuntu 14.04LTS (but not in 12.04LTS).
It's probably not a bad idea, but given the level of pruning that's been needed already to keep down the false positive rate, I'm nervous about a larger "general" corpus.
-Kees
-- Kees Cook Chrome OS Security
| |