Re: change_rev_prop atomicity

On 2/18/07, David Glasser <glasser@mit.edu> wrote:
> Malcolm,
>
> revs-txns.c currently just contains svn_fs_fs__revision_prop,
> svn_fs_fs__change_rev_prop, svn_fs_fs__get_txn_ids,
> svn_fs_fs__txn_prop, and svn_fs_fs__begin_txn. The second should move
> into fs_fs.c, for locking reasons. Is there any good reason not to
> just move them all into fs_fs.c and eliminate revs-txns.[ch] while I'm
> at it? It is much smaller than libsvn_fs_base's revs-txns.c.

... because if that's a reasonable decision, then I have a patch that
does that and a second patch that adds a lock around
svn_fs_fs__change_rev_prop that are ready to push.

--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