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

["Jörn Engel" <joern@logfs.org>]

On Tue, 25 March 2008 13:24:14 +0100, Ingo Molnar wrote:
> 
> The current visual inconsistency between subsystems makes the Linux 
> kernel appear rather unpleasant and unprofessional to new kernel 
> developers. This is not just embarrasing to us (we want to write the 
> best OS on the planet), it is also actively harmful: such a "consistent 
> style does not matter" stance turns away people who have taste and tends 
> to attract people who have no taste - which i'm sure you'll agree with 
> results in a deadly spiral if it gets strong enough.

I disagree with that assertion.  My favorite example of where
CodingStyle has gone too far is this:
	for (i=0; i<10; i++)
While the official document demands four extra spaces, I _hate_ them.
Whitespace offers visual grouping.  The lack of whitespace around the
binary operators emphasizes that one kind of grouping is stonger than
another.  Ever since this binary operator testament was added to our
Holy Canon, I started violating the coding style on purpose.  Imo this
is beyond silly.

So do I have bad taste and should I leave the kernel in favor of someone
else with better taste that is currently turned away by me?  Maybe.
Show me that person and I'll consider gardening.  Until then I'll
continue to violate the style.  Just to spite the fundamentalist
movement.

> So to turn around the argument: could you give me any reason why 
> differing coding style between subsystems, _often in blatant violation 
> of Documentation/CodingStyle_, is somehow "good" for Linux in the long 
> run? I listed numerous first-hand advantages that style consistency 
> brings and i listed numerous disadvantages created by inconsistency. So 
> i'm waiting for the list of counter-arguments - there _must_ be some 
> objective ones, besides the obvious "kernel old-timers are lazy to 
> change their ways" argument =B-)

When you reject useful patches based on "this is not our preferred
style", you piss people off.  That is a significant reason why people
choose to spend their time elsewhere.  In certain cases having people
abandon the kernel may be a net gain, in many it is a loss.

So unless you are willing to maintain every single driver in the kernel,
pissing all the maintainers off that happen to disagree with the
canonical style - if only in detail - is not a good recipe.

Don't get me wrong, I certainly see advantages in checkpatch and keeping
the style consistent.  But there are limits, where the gains no longer
justify the cost.  And the limits will never be clearly defined.  Are
some variable names better than others - sure.  Can you write a rule for
checkpatch to ensure good names - hardly.  And yet, variable names are
part of the style.

> These style differences are certainly not "wrong enough" to 
> inconvenience or displace an active maintainer (and i never made that 
> point)

It seems we both agree then.  And for the record, your mail could easily
be interpreted as if you had made that point.  Thanks for clarifying
things.

Jörn

-- 
I don't understand it. Nobody does.
-- Richard P. Feynman