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

[JÃrn Engel]

On Wed, 26 March 2008 12:23:11 +0100, Ingo Molnar wrote:
> * JÃrn Engel <joern@xxxxxxxxx> wrote:
> 
> > 
> > Then I'll happily ignore it.  Not having the space gains me one 
> > column. It is absolutely minimal, sure.  But when the alternative is 
> > based on pure whim...
> 
> you seem to be confused here. Consistency is not a 'whim', and it is 
> often just about "arbitrary" choices that also have some ergonomic 
> component as well (but sometimes not). The following form:

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/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/