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

Re: svn commit: r19782 - in branches/changelist-feature/subversion: include libsvn_client svn

From: Garrett Rooney <rooneg_at_electricjellyfish.net>
Date: 2006-05-23 20:21:50 CEST

On 5/23/06, sussman@tigris.org <sussman@tigris.org> wrote:

> +
> +/**
> + * Beginning at @a root_path, do a recursive walk of a working copy
> + * and discover every path which belongs to @a changelist_name.
> + * Return the list of paths in @a *paths. If no matching paths are
> + * found, return an empty array.
> + *
> + * @a cancel_func/cancel_baton are optional. If non-NULL, poll them
> + * for user cancellation during the recursive walk.
> + *
> + * @since New in 1.5.
> + */
> +svn_error_t *
> +svn_client_retrieve_changelist(apr_array_header_t **paths,
> + const char *changelist_name,
> + const char *root_path,
> + svn_cancel_func_t cancel_func,
> + void *cancel_baton,
> + apr_pool_t *pool);
> +

In the past we've had to go from functions like this that gather up
all their return values and pass them back in an array to streamy
interfaces that use a callback/baton pair. Would it make sense to
just start this with the streamy version? I mean you can build the
current interface on top of a callback driven version, but not the
other way around...

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue May 23 20:22:34 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.