[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r32628 - trunk/subversion/tests/libsvn_fs

From: Karl Fogel <kfogel_at_red-bean.com>
Date: Fri, 22 Aug 2008 19:42:56 -0400

"Greg Stein" <gstein_at_gmail.com> writes:
> Yah. I looked briefly in the hacking guide. Not sure it ever made it
> there, but I think I recall discussing it with sussman, and he agreed.
>
> The reason that I prefer it is:
>
> const char *var1, **var2;
>
> It is sometimes difficult to see that var2 is different. One per line
> tends to make stuff like the above clearer.

Yeah... I kind of agree in this case, but am wary of loading us down
with even more coding conventions. We're bordering on being a
"high-maintenance project" I think :-), and I'd rather see us relax
about some conventions than add yet more.

YMMV, though, and if you want to add something to hacking.html, I won't
object (beyond the above statement of preference).

-K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-08-23 01:43:10 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.