Re: change_rev_prop atomicity

On 2/19/07, Malcolm Rowe <malcolm-svn-dev@farside.org.uk> wrote:
> On Sun, Feb 18, 2007 at 02:59:41PM -0500, David Glasser wrote:
> > >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.
> >
>
> I think that's a reasonable thing to do - and we will probably find that
> there's some additional refactoring we can do after we've done that.
> For example, svn_fs_fs__get_txn() can become a static function.

OK, I killed revs-txns in r23439 and (hopefully) fixed the race
condition in r23440.

I didn't do the refactoring you just mentioned.

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