Re: [PATCH 02/27] drivers/net: fix sparse warnings: make do-whilea compound statement

From: Linus Torvalds
Date: Tue Dec 23 2008 - 21:10:50 EST




On Wed, 24 Dec 2008, Krzysztof Halasa wrote:
>
> People learn, or should, through the life :-)

Sure. But you should learn about the things that matter - not learn to
avoid the stupid pitfalls that come from confusingly doing things so that
they visually look similar even when they do different things.

So don't make people learn by putting traps in their face. That just
wastes everybodys time.

> I'm not sure being common or less common does matter here much.
>
> OTOH I think it's pretty common. Approx as common as while (x) y is,
> isn't it?

I doubt it. It certainly wasn't in the kernel. When we added the sparse
warning, I think we got a couple of hits.

Anyway, not worth discussing. The fact is, the kernel does not accept do
while without braces. I told you why. You can ignore it. I'll ignore you.

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