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

Re: svn commit: r27607 - in trunk/subversion: include libsvn_fs libsvn_fs_base libsvn_fs_fs libsvn_repos libsvn_subr

From: Blair Zajac <blair_at_orcaware.com>
Date: 2007-11-06 02:04:06 CET

glasser@tigris.org wrote:
> Author: glasser
> Date: Mon Nov 5 16:45:46 2007
> New Revision: 27607
> Log:
> Add fs- and repos-level APIs to change multiple transaction properties
> atomically.

> +/** Change, add, and/or delete transaction property values in
> + * transaction @a txn. @a props is an array of <tt>svn_prop_t</tt>
> + * elements. Do any necessary temporary allocation in @a pool.
> + *
> + * @since New in 1.5.
> + */
> +svn_error_t *svn_fs_change_txn_props(svn_fs_txn_t *txn,
> + apr_array_header_t *props,
> + apr_pool_t *pool);

This description isn't too clear to me. Could you add a section describing that
this is effectively an efficient way of calling svn_fs_change_txn_prop()
multiple times and it has the same behavior to remove elements (to pass NULL)?

I had to check the code to see that's what it was doing.

BTW, the reason I'm asking this new method was that this could have complete
replacement schematics in that props is the complete list of new props, but
that's not what it does.


To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Nov 6 02:04:17 2007

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.