Re: [PATCH] CodingStyle: proscribe do-while without braces.

From: Stefan Richter
Date: Fri Jul 27 2007 - 13:14:27 EST


Andrew Morton wrote:
> On Fri, 27 Jul 2007 09:41:01 +0200 Stefan Richter <stefanr@xxxxxxxxxxxxxxxxx> wrote:
...
>> SubmitChecklist already tells us to check with sparse before submission.
>
> rofl. First we have to talk everyone into checking their stuff with gcc.

Ah, I forgot --- Real Submitters don't compile their code before
submission. But wait, they also don't track CodingStyle.

Seriously though: Precise rules are a priori welcome. But the more
rules there are, the less of them we (authors, reviewers) can memorize.
If we go for having many rules, then we need to rely more on checkpatch
and sparse. When growing into a catch-all rule book, the utility of
CodingStyle will shift from an author's guide to a specification for
code checking tools.
--
Stefan Richter
-=====-=-=== -=== ==-==
http://arcgraph.de/sr/
-
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/