lkml.org 
[lkml]   [2003]   [Mar]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: Those ruddy punctuation fixes
From
Date
On Wed, 2003-03-05 at 05:20, Dave Jones wrote:
> On Wed, Mar 05, 2003 at 11:10:15AM +0000, Russell King wrote:
>
> > Could we stop fix^wbreaking this stuff please. GCC 3.2.2:
> > ...
> > include/asm/proc-fns.h:128:39: missing terminating ' character
>
> 100% agreed. People really are going too far IMO.
> Given that most people will never read those comments, the
> effort would be much better spent proof-reading/correcting
> documentation than comments.
>
> The "its just a spelling mistake, it cant break the build!"
> mantra is also way off base. If you touch a .c/.h file, you
> introduce the possibility of breakage.

That is why I was very careful with my its -> it's patch.
In the two files where an extra apostrophe would have broken
the build, I changed its to it is. Why not just leave it alone?
Because some well-meaning spelling fixer may come along in the
future and break it, just like in proc-fns.h.

>
> The cant -> can't pedantry is an example of just how extremely
> silly these are getting. Is there really someone who sees
> "cant" and doesn't understand what it could mean?
> It just pedantic masturbation AFAICS.
>
> Dave

It's too bad that the cant -> can't change is tarred with the same brush
as thresold -> threshold and asociation -> association. My belief is
that those kind of fixes have these benefits:

The comments are more readable.
The source can be grepped more accurately.
Correct spelling sets a good example.
Incorrect spelling sets a bad example and proliferates.

Why all the fixes now? Because it's a window of opportunity. Linus
seems more comfortable with accepting these kind of changes now than at
other times.

If we don't get these changes in now, expect to have this same
conversation around 2.9.50 or so. (Assuming that 3.0 will follow 2.6)

Cheers,
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/

\
 
 \ /
  Last update: 2005-03-22 13:33    [W:0.085 / U:0.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site