git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Linus and whitespace

From
A Large Angry SCM <gitzilla@gmail.com>
Date
Feb 1, 2009, 22:48 UTC
Message-ID
<498626D3.1020202@gmail.com>
In-Reply-To
<87fxix3hub.fsf@jidanni.org>
jidanni@jidanni.org wrote:
Show 33 quoted lines
> 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?

Previous: jidanni@jidanni.orgNext: Linus Torvalds
Message 2 of 4 in “Linus and whitespace”
  1. jidanni@jidanni.orgFeb 1, 2009
  2. A Large Angry SCMFeb 1, 2009
  3. Linus TorvaldsFeb 1, 2009
  4. Johannes GilgerFeb 2, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.