git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Linus and whitespace
@ 2009-02-01 22:36 jidanni
  2009-02-01 22:48 ` A Large Angry SCM
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: jidanni @ 2009-02-01 22:36 UTC (permalink / raw)
  To: torvalds; +Cc: git

Gentlemen, I couldn't help but notice a certain Linus B. Torvalds is
producing mail with trailing whitespace.

I recall one should (linux-doc-*/Documentation/CodingStyle*:)
"Get a decent editor and don't leave whitespace at the end of lines."

Indeed, with my GNU Emacs show-trailing-whitespace X-ray
eyespecs on all day, whitespace becomes screaming purple space, so you
can't blame me for noticing.

Anyway, I have analyzed his messages,
(User-Agent: Alpine 2.00 (LFD 1167 2008-08-23))
and discovered that the whitespace phenomenon almost always occurs if
the lines are more than 60 characters long, the "danger zone", we might say.

$ perl -nwle 'print length if / $/' recent_linus_postings|
sort|uniq -c|sort -k 2nr
      1 77
      1 76
     17 75
     17 74
     11 73
     18 72
     12 71
      6 70
      8 69
      6 68
      2 67
      1 66
      2 64
      1 63
      1 40
      9 2

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Linus and whitespace
  2009-02-01 22:36 Linus and whitespace jidanni
@ 2009-02-01 22:48 ` A Large Angry SCM
  2009-02-01 22:56 ` Linus Torvalds
  2009-02-02  7:22 ` Johannes Gilger
  2 siblings, 0 replies; 4+ messages in thread
From: A Large Angry SCM @ 2009-02-01 22:48 UTC (permalink / raw)
  To: jidanni; +Cc: torvalds, git

jidanni@jidanni.org wrote:
> Gentlemen, I couldn't help but notice a certain Linus B. Torvalds is
> producing mail with trailing whitespace.
> 
> I recall one should (linux-doc-*/Documentation/CodingStyle*:)
> "Get a decent editor and don't leave whitespace at the end of lines."
> 
> Indeed, with my GNU Emacs show-trailing-whitespace X-ray
> eyespecs on all day, whitespace becomes screaming purple space, so you
> can't blame me for noticing.
> 
> Anyway, I have analyzed his messages,
> (User-Agent: Alpine 2.00 (LFD 1167 2008-08-23))
> and discovered that the whitespace phenomenon almost always occurs if
> the lines are more than 60 characters long, the "danger zone", we might say.
> 
> $ perl -nwle 'print length if / $/' recent_linus_postings|
> sort|uniq -c|sort -k 2nr
>       1 77
>       1 76
>      17 75
>      17 74
>      11 73
>      18 72
>      12 71
>       6 70
>       8 69
>       6 68
>       2 67
>       1 66
>       2 64
>       1 63
>       1 40
>       9 2

CodingStyle applies to *patches* not to discussion. In how many of those 
emails you analyzed were there any trailing whitespace in a patch?

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Linus and whitespace
  2009-02-01 22:36 Linus and whitespace jidanni
  2009-02-01 22:48 ` A Large Angry SCM
@ 2009-02-01 22:56 ` Linus Torvalds
  2009-02-02  7:22 ` Johannes Gilger
  2 siblings, 0 replies; 4+ messages in thread
From: Linus Torvalds @ 2009-02-01 22:56 UTC (permalink / raw)
  To: jidanni; +Cc: git



On Mon, 2 Feb 2009, jidanni@jidanni.org wrote:
>
> Gentlemen, I couldn't help but notice a certain Linus B. Torvalds is
> producing mail with trailing whitespace.

alpine does it when word-wrapping my messages. 

But I don't word-wrap my _patches_. Those I just insert, and alpine gets 
it right.

			Linus

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Linus and whitespace
  2009-02-01 22:36 Linus and whitespace jidanni
  2009-02-01 22:48 ` A Large Angry SCM
  2009-02-01 22:56 ` Linus Torvalds
@ 2009-02-02  7:22 ` Johannes Gilger
  2 siblings, 0 replies; 4+ messages in thread
From: Johannes Gilger @ 2009-02-02  7:22 UTC (permalink / raw)
  To: git

On 2009-02-01, jidanni@jidanni.org <jidanni@jidanni.org> wrote:
> Gentlemen, I couldn't help but notice a certain Linus B. Torvalds is
> producing mail with trailing whitespace.

Boy, do you have too much free time on your hands...

Greetings,
Jojo

-- 
Johannes Gilger <heipei@hackvalue.de>
http://hackvalue.de/heipei/
GPG-Key: 0x42F6DE81
GPG-Fingerprint: BB49 F967 775E BB52 3A81  882C 58EE B178 42F6 DE81

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2009-02-02  7:23 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-02-01 22:36 Linus and whitespace jidanni
2009-02-01 22:48 ` A Large Angry SCM
2009-02-01 22:56 ` Linus Torvalds
2009-02-02  7:22 ` Johannes Gilger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).