Re: [PATCH 109/148] include/asm-x86/serial.h: checkpatch cleanups - formatting only

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Ingo Molnar <mingo@...>
Cc: Andy Whitcroft <apw@...>, David Miller <davem@...>, <jirislaby@...>, <viro@...>, <joe@...>, <tglx@...>, <linux-kernel@...>
Date: Wednesday, March 26, 2008 - 7:41 am

On Wed, 26 March 2008 12:23:11 +0100, Ingo Molnar wrote:

I have an objective reason to prefer one over the other.  Your only
reason is consistency.  If consistency was everything, we might as well
let the often-abused 1000 monkeys type up a coding style document and
stick to that.

But most of our rules exist for reasons beyond mere consistency.
Following the rules, on average, makes the code better.  In particular
the "less" rules (fewer lines, less indentation, etc.) result in more
code fitting any arbitrary editor window.  Which means more control flow
our mind can ponder about without scrolling.

Do you have a non-consistency based reason to prefer the longer version?
If not, then we should settle on the short version, which does have a
minimal advantage.

Jörn

-- 
Joern's library part 13:
http://www.chip-architect.com/
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH 109/148] include/asm-x86/serial.h: checkpatch cle..., Christoph Hellwig, (Wed Mar 26, 7:09 am)
[patch] bkl2mtd: cleanup, Ingo Molnar, (Wed Mar 26, 6:14 am)
Re: [patch] bkl2mtd: cleanup, Al Viro, (Wed Mar 26, 6:48 am)
Re: [patch] bkl2mtd: cleanup, Ingo Molnar, (Wed Mar 26, 7:10 am)
Re: [patch] bkl2mtd: cleanup, Joe Perches, (Wed Mar 26, 12:30 pm)
Re: [patch] bkl2mtd: cleanup, Jiri Slaby, (Wed Mar 26, 7:14 am)
Re: [patch] bkl2mtd: cleanup, Ingo Molnar, (Wed Mar 26, 7:02 am)
Re: [patch] bkl2mtd: cleanup, Ingo Molnar, (Wed Mar 26, 7:00 am)
Re: [patch] bkl2mtd: cleanup, Jörn, (Wed Mar 26, 6:57 am)
Re: [PATCH 109/148] include/asm-x86/serial.h: checkpatch cle..., Jörn, (Wed Mar 26, 7:41 am)