Re: svn commit: r22599 - in branches/multiple-moves/subversion: include libsvn_client svn

On 12/7/06, hwright@tigris.org <hwright@tigris.org> wrote:
> On the multiple moves branch:
> Merge the svn_client_[copy,move] and svn_client_[copy,move]_into APIs. The
> idea of the new APIs is to Do The Right Thing when given a list of source paths
> and a destination path. The new APIs also have a flag to preserve compatibility
> with the changes introduced in r18315 to fix issue 2188.

The more I think about it, the more it seems like two separate APIs is
better than one API with a boolean that entirely changes its
semantics. For example, I found the documentation for the separated
_into function easier to understand than the documentation for the
combined function. (I'm still not sure what the combined API should
do if you pass a multiple-element array but set copy_as_child to
FALSE.) (Also, in the common case of a single-file-copy/move, this
requires extra code to create an array.)

--dave

-- 
David Glasser | glasser_at_mit.edu | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org