Re: [PATCH] Authz support in Svnserve

On Fri, 2005-08-26 at 00:29 +0200, Branko Èibej wrote:
> And woldn't it be nice if "b" were called "baton" instead.
> Single-character variables make my froodleprog itch.

We use single-character or two-character variable names for batons all
over the code base.

If Subversion were in C++ or Java, baton fields would be class
variables, and we wouldn't have a "b->" or "eb->" in front of them at
all. Since this is C, we have to do that work ourselves, but it makes
sense to minimize the amount of syntactic fluff necessary to do so.

Admittedly, too much of this makes our code look like early BSD code,
which (to me) is completely impenetrable. But I don't think doing this
for batons is too much. Longer variable names do not always improve
readability, "i" is sometimes better than "loop_iteration_count".

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Aug 26 18:34:02 2005