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

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

From: Greg Hudson <ghudson_at_MIT.EDU>
Date: 2006-06-12 23:06:33 CEST

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

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.