Re: Finding typos using codespell

From: J. Hellenthal <jhell_at_DataIX.net>
Date: Tue, 19 Apr 2011 20:53:49 -0400
On Tue, Apr 19, 2011 at 08:17:37PM -0400, J. Hellenthal wrote:
>On Tue, Apr 19, 2011 at 02:22:10PM +0200, Ulrich Spörlein wrote:
>>On Tue, 19.04.2011 at 12:15:00 +0100, Bruce Cran wrote:
>>> There's a new tool that can be used to find spelling mistakes in code: codespell 
>>> from http://www.politreco.com has already been used to find mistakes in both 
>>> Linux and LLVM. I ran it on sys/ and it found lots of potential typos - the 
>>> full diff (which I know does contain some incorrect changes) can be found at 
>>> http://www.cran.org.uk/~brucec/freebsd/codespell_sys.diff .
>>
>>Needs manual review, but I think this is very useful for the spelling
>>nazi challenged among us :-)
>>
>>False positive, e.g.:
>> {
>>-       struct ccb_immed_notify *inot;
>>+       struct ccb_immed_notify *into;
>>        mpt_tgt_state_t *tgt;
>>
>>Cheers,
>>Uli
>
>As mentioned here already changes that I have noticed are:
>
>(Upper || lower case):
>inot -> into
>lsat -> last
>inbetween -> between
>ALOT -> A LOT
>boundry -> boundary	"This one seems to have corrected itself."
>virtualY -> virtually
>ID -> DID
>lastr -> last
>*Any uuencoded file*
>
>This was just a review of the functional changes. The no-op changes I
>just filtered out of the patch to quickly catch the "Bad Thing" changes
>above.
>
>It might just be better to build a list of files with spelling errors
>and anyone with the next commit to any of those files just runs it
>through aspell/ispell or whatever they like rather than causing this
>much of a slush.
>
>All in all good find Bruce!.
>

Meant to attach the final list I came up with for further use if needed
but forgot on send...

Here it is.

-- 

 Regards,
 J. Hellenthal
 WWJD


Received on Tue Apr 19 2011 - 22:53:56 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:13 UTC