Re: [PATCH] kernel source spellchecker

From: Steven Cole (elenstev@mesatop.com)
Date: Sat Mar 01 2003 - 22:29:47 EST


On Sat, 2003-03-01 at 19:08, Dan Kegel wrote:
[snipped]
>
> This corrections file is probably good enough to actually use.
> I'm running it against linux-2.5.63-bk5 now...
> - Dan
[snippage]
> Pseudo=Psuedo

Hmm, psuedo didn't get caught. Is psuedo code particularly smooth?

Anyway, looking beyond comments only may catch real bugs.
I found this a few days ago whilst looking for commonly
misspelled words, a much weaker technique than your spellchecker.

[steven@spc5 linus-2.5]$ bk export -tplain ../linux
[steven@spc5 linus-2.5]$ cd ../linux
[steven@spc5 linux]$ find . -type f | xargs grep psuedo
./arch/ppc64/kernel/iSeries_IoMmTable.h:/* allocated the psuedo I/O Address. */
./drivers/base/platform.c: * platform.c - platform 'psuedo' bus for legacy devices
./drivers/scsi/aic7xxx/aic7xxx.seq: * use byte 27 of the SCB as a psuedo-next pointer and to thread a list
./drivers/scsi/g_NCR5380.c: * Perform a psuedo DMA mode read from an NCR53C400 or equivalent
./drivers/scsi/g_NCR5380.c: * Perform a psuedo DMA mode read from an NCR53C400 or equivalent
./drivers/video/skeletonfb.c: * no color palettes are supported. Here a psuedo palette is created
./drivers/video/anakinfb.c: fb_info.psuedo_palette = colreg;
----------------------------------------^^^^^^
This shouldn't even compile.

[steven@spc5 linux]$ find . -name "*.h" | xargs grep pseudo_palette
./drivers/video/i810/i810.h: u32 pseudo_palette[17];
./include/linux/fb.h: void *pseudo_palette; /* Fake palette of 16 colors and

Yep, a mistake all right. Adding the listed author to the cc list.

Steven

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Fri Mar 07 2003 - 22:00:17 EST