Re: [PATCH] fix similar typos to successfull, v2



On Tue, 25 Nov 2008 21:49:13 -0200
Fernando Carrijo <fcarrijo@xxxxxxxxxxxx> wrote:

On Tue, 2008-11-25 at 14:53 -0800, Andrew Morton wrote:
On Tue, 25 Nov 2008 01:44:13 +0800
Coly Li <coyli@xxxxxxx> wrote:

When I review ocfs2 code, find there are 2 typos to "successfull". After doing grep "successfull "
in kernel tree, 22 typos found totally -- great minds always think alike :)

Mutter. I have a super-strict (lol) policy of "I don't take patches
which fix typos/spellos in comments", because we have so many such
errors, and I'd get buried in silly patches. But I do take patches
which fix documentation and user-visible printks, etc.

Since you started the thread, I feel less guilty to talk about the
existence of tons of another kind of typo spread around the kernel
sources: null statements created by duplicated semi-colons at the end of
statements. Something like foo();;

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



Relevant Pages

  • Re: [PATCH] fix similar typos to successfull, v2
    ... in kernel tree, 22 typos found totally -- great minds always think alike:) ... I have a super-strict policy of "I don't take patches ... which fix typos/spellos in comments", because we have so many such ...
    (Linux-Kernel)
  • [PATCH 0/2] UML - two small fixes
    ... These two patches are 2.6.16 material. ... by an earlier patch, and fix a couple of typos. ...
    (Linux-Kernel)
  • 2.6.5-rc2-tiny1 for small systems
    ... This is the latest release of the -tiny kernel tree. ... the patches and their config options in preparation for merging ... send the line "unsubscribe linux-kernel" in ...
    (Linux-Kernel)
  • 2.6.5-tiny1 for small systems
    ... This is the latest release of the -tiny kernel tree. ... are things like embedded systems, small or legacy desktop folks, and ... the patches and their config options in preparation for merging ... 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/ ...
    (Linux-Kernel)
  • Re: [PATCH] add a trivial patch style checker v2
    ... checker and include it in the kernel tree. ... +# Licensed under the terms of the GNU GPL License version 2 ... Fix checkpatch.pl name in usage template ...
    (Linux-Kernel)