lkml.org 
[lkml]   [2014]   [Aug]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] checkpatch: look for common misspellings
From
Date
On Fri, 2014-08-22 at 23:00 +0000, Elliott, Robert (Server Storage)
wrote:
> > From: linux-kernel-owner@vger.kernel.org [mailto:linux-kernel-
> > owner@vger.kernel.org] On Behalf Of Joe Perches
> ...
> > +chang||change
>
> Although there are at least 2 misuses in the current kernel,
> that's also a fairly common name.

True. Thanks.

That one should likely be removed.
There are dozens of Chang name uses in the kernel.

I'm not attached at all to any of these btw.

Any others you think should not be used or
should be added?

One I thought kind of iffy was

contant||contact

Maybe that should be content instead.

Another thing the script might do is to check
if the $typo match is all upper case and then
substitute an all upper case $typo_fix.

It's also kind of noisy when run using -f,
so maybe this test should be --strict check with
files and a warning with patches.




\
 
 \ /
  Last update: 2014-08-23 02:01    [W:0.042 / U:0.144 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site