Messages in this thread | | | Date | Mon, 11 Aug 2003 10:23:33 -0700 | From | Larry McVoy <> | Subject | Re: [PATCH] CodingStyle fixes for drm_agpsupport |
| |
On Mon, Aug 11, 2003 at 01:15:58PM -0400, Jeff Garzik wrote: > > if (expr) statement; // OK > > The test and the statement run together visually, which is it is > preferred to put the statement on the following line.
Nah.
if (!p) return (whatever); if (foo) { statement; } else { statement; statement; } if (!p) return (whatever);
Perfectly readable. We have a few hundred thousand lines of code written like this and I review all of it. I suspect that I do more reviewing than 99% of the people on this list which makes my opinion count more because anything that makes my tired eyes absorb the info faster is a good thing. Same for your eyes when you get to my age.
I also make people do
if ((a <= B) || (c >= d)) { xxx }
even though I know, if I think about it, what the precedence is. It doesn't matter that I know or you know, what matters is the number of lines of code a day you can correctly review. Anything that helps that means that you are helping people make the source base better. Try reading 30K lines of diffs at one sitting and tell me again that I'm wrong. If you do, bump it up to 60K lines :)
> > if (!pointer) return (-EINVAL); > > > >Short, sweet, readable, no worries. > > return is not a function ;-)
See, there is that age thing again. Think V6. And it is sort of a function, it unravels the stack frame. -- --- Larry McVoy lm at bitmover.com http://www.bitmover.com/lm - 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/
| |