Re: Please fix or revert: [PATCH] checkpatch: add --strict tests forbraces, comments and casts

From: Ingo Molnar
Date: Wed Apr 25 2012 - 04:42:16 EST



(Sorry about the late reply, was busy with other things.)

* David Miller <davem@xxxxxxxxxxxxx> wrote:

> From: Ingo Molnar <mingo@xxxxxxxxxx>
> Date: Mon, 16 Apr 2012 22:26:54 +0200
>
> > If you don't "see" it as ugly it's simply because your brain
> > is not wired up to see 3D/2D geometry as a significant
> > source of information.
>
> I'm sorry that I'm so handicapped that I cannot match your
> visual capabilitites.

I genuinely think that it's the other way around: if what I say
above is true then *you* have the superior vision for coding.

Geometric properties of code are a distraction really, they
don't matter to the end result.

So in reality keeping comments balanced helps *my* disability:
my inability to ignore functionally irrelevant patterns of
characters...

So I am asking *you* to do me a favor: please understand my
perspective and insert a bit more whitespace to keep code
readable for people like me.

( The wider question is, what is the contribution weighted
proportion of the two groups of kernel developers and what is
the disutility and utility of the two variants. It's a
difficult balancing task and there's no perfect solution. )

Thanks,

Ingo
--
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/