mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@osdl.org>
To: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: dwmw2@infradead.org, linux-kernel@vger.kernel.org
Subject: Re: - add-pselect-ppoll-system-call-implementation-tidy.patch removed from -mm tree
Date: Wed, 18 Jan 2006 22:30:39 -0800	[thread overview]
Message-ID: <20060118223039.1d9dfe64.akpm@osdl.org> (raw)
In-Reply-To: <20060119171708.7f856b42.sfr@canb.auug.org.au>

Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> Documentation/CodingStyle says:
> 
>  The limit on the length of lines is 80 columns and this is a hard limit.
> 
>  Statements longer than 80 columns will be broken into sensible chunks.
>  Descendants are always substantially shorter than the parent and are placed
>  substantially to the right. The same applies to function headers with a long
>  argument list. Long strings are as well broken into shorter strings.

That's pretty stern.

I'd be happy with a 96-col standard, or 100 or whatever - it's more
convenient and I use twin 20" guns.  But other people have different
hardware constraints and different work practices, so they want 80 cols. 
If we're going to get down and change the standard then OK, let's have that
bunfight.  But while there's a standard we should stick to it so we don't
screw over the people who like to use standard-sized xterms.

And yes, some editors can do sideways-scrolling to make wider-than-80
acceptable in an 80-col window.  But other people's setups don't do that,
and the cost to those people of wrappy code is higher than the cost of
looking at standardly-laid-out code to fancy-editor users.

So the lowest common denominator wins, because they hurt more than anyone
else if we go outside 80-cols.  I use 80-col xterms precisely for this
reason: so that the code which goes in will look OK to those users.

  parent reply	other threads:[~2006-01-19  6:31 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200601190052.k0J0qmKC009977@shell0.pdx.osdl.net>
2006-01-19  5:21 ` David Woodhouse
2006-01-19  6:17   ` Stephen Rothwell
2006-01-19  6:24     ` David Woodhouse
2006-01-19  6:30     ` Andrew Morton [this message]
2006-01-19  6:40       ` David Woodhouse
2006-01-19  6:36     ` David S. Miller
2006-01-19  6:47       ` Trond Myklebust
2006-01-19  7:02         ` Andrew Morton
2006-01-19  7:18           ` David Woodhouse
2006-01-19  8:09           ` David S. Miller
2006-01-19 15:51       ` James Morris
2006-01-20  2:17       ` Stephen Rothwell
2006-01-19  9:58     ` Alan Cox
2006-01-19 15:59       ` Jens Axboe
2006-01-20  8:33         ` David Woodhouse
2006-01-20  8:44           ` Andrew Morton
2006-01-20  8:59             ` David Woodhouse
2006-01-20 10:01               ` Eric Dumazet
2006-01-23  5:25                 ` David Woodhouse
2006-01-20 23:44               ` Adrian Bunk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060118223039.1d9dfe64.akpm@osdl.org \
    --to=akpm@osdl.org \
    --cc=dwmw2@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sfr@canb.auug.org.au \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®