Re: changelist API : enhancement request

Daniel Rall wrote:
> Why are there printf()'s in the library APIs? We can't ship this or
> anything like it without that TODO getting fixed...

Agreed. This doesn't prevent the API change, though, does it?

> Index: subversion/libsvn_client/changelist.c
> ===================================================================
> --- subversion/libsvn_client/changelist.c (revision 23045)
> +++ subversion/libsvn_client/changelist.c (working copy)
> ...
> svn_error_t *
> -svn_client_set_changelist(const char *path,
> +svn_client_set_changelist(const apr_array_header_t *paths,
> const char *changelist_name,
> svn_client_ctx_t *ctx,
> apr_pool_t *pool)
> {
> - SVN_ERR(svn_wc_set_changelist(path, changelist_name, pool));
> + int i;
> + apr_pool_t *iterpool = svn_pool_create(pool);
>
> - /* ### TODO(sussman): create new notification type, and send
> - notification feedback. See locking-commands.c. */
> - if (changelist_name)
> - printf("Path '%s' is now part of changelist '%s'.\n",
> - path, changelist_name);
> - else
> - printf("Path '%s' is no longer associated with a changelist'.\n", path);
> + for (i = 0; i < paths->nelts; i++)
> + {
> + const char *path = APR_ARRAY_IDX(paths, i, const char *);
>
> + /* Check for cancellation */
> + if (ctx->cancel_func)
> + SVN_ERR(ctx->cancel_func(ctx->cancel_baton));
> +
> + svn_pool_clear(iterpool);
> + SVN_ERR(svn_wc_set_changelist(path, changelist_name, iterpool));
> +
> + /* ### TODO(sussman): create new notification type, and send
> + notification feedback. See locking-commands.c. */
> + if (changelist_name)
> + printf("Path '%s' is now part of changelist '%s'.\n",
> + path, changelist_name);
> + else
> + printf("Path '%s' is no longer associated with a changelist'.\n", path);
> + }
> +
> + svn_pool_destroy(iterpool);
> +
> return SVN_NO_ERROR;
> }