Re: janitorial PATCH: 2.4: nvram.c Lindent

Linus Torvalds (torvalds@transmeta.com)
Tue, 30 Jul 2002 22:52:53 +0000 (UTC)


In article <3D47170E.20003@sun.com>, Tim Hockin <thockin@sun.com> wrote:
>
>This patch is pretty simple: It runs drivers/char/nvram.c through
>Lindent, with a few manual cosmetics on top. I'm sending this now
>because it makes my follow-up patch to this file easier :)

Hmm.

If you're doing these kinds of Lindent changes, you might as well also
fix another non-linuxism:

return (x); -> return x;

I don't know why some people seem to think that "return" is a function
with an argument..

I guess that one isn't mentioned in the CodingStyles thing. I'm lazy.
Bad Bad Linus.

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