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

Re: [PATCH] Re: The new svn_wc_adm_*_depth functions.

From: <kfogel_at_collab.net>
Date: 2004-03-11 16:51:29 CET

Greg Hudson <ghudson@MIT.EDU> writes:
> With one set of functions, we're adding foo_depth() variants which
> include a depth argument.
>
> With another set of functions, we didn't want to add foo_pool() variants
> because that looks dumb, so we're adding foo2() variants instead.
>
> I would like us to be consistent.

Good point. A question that impinges on this:

Are we planning to keep these suffixes in 2.0? Or will we change the
new names to the old names (just keep the new APIs) when we cross the
2.0 line?

If we're going to fall back to the old names, then the "2" variants
might make the most sense.

IMHO, we should do this. By resetting our APIs back to the
names-of-first-choice, we avoid names growing arbitrarily long (think
3.0).

Thoughts?

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Mar 11 17:58:39 2004

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.