Re: svn commit: r20033 - in branches/merge-tracking/subversion: include libsvn_repos svnserve

On Mon, 2006-06-12 at 12:13 -0700, Daniel Rall wrote:
> The first nested tuple should be a list. I think I need to add an
> ellipses to indicate that.
>
> Also, "rev" should be an optional parameter (the server defaults to
> "youngest" -- the protocol doc might want to indicate that, too).
> Does "rev" need to be placed in a tuple, or can it simply be preceded
> by a question mark?

With all due respect, there's nothing unique about this request; you can
just look at how other commands handle similar constructs.

You're correct on the first point. On the second point, use the same
optional-tuple syntax used by every other command, which puts the
argument inside a tuple. That way the command can be extended to
include more optional arguments.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jun 12 23:08:06 2006