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

Re: svn commit: r22610 - in trunk/subversion: include libsvn_fs libsvn_fs_base libsvn_fs_fs libsvn_repos mod_dav_svn tests/libsvn_fs

From: Malcolm Rowe <malcolm-svn-dev_at_farside.org.uk>
Date: 2006-12-08 19:09:44 CET

Hi Mike,

On Fri, Dec 08, 2006 at 08:16:31AM -0800, cmpilato@tigris.org wrote:
> * subversion/libsvn_repos/replay.c
> (svn_repos_replay2): use new svn_fs_txn_root_base_revision() instead
> of the roundabout way.
>

> Modified: trunk/subversion/libsvn_repos/replay.c
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/libsvn_repos/replay.c?pathrev=22610&r1=22609&r2=22610
> ==============================================================================
> --- trunk/subversion/libsvn_repos/replay.c (original)
> +++ trunk/subversion/libsvn_repos/replay.c Fri Dec 8 08:16:30 2006
> @@ -711,26 +711,20 @@
>
> if (send_deltas)
> {
> + svn_revnum_t compare_root_revision = SVN_INVALID_REVNUM;
> + if (svn_fs_is_revision_root(root))
> + compare_root_revision = svn_fs_revision_root_revision(root) - 1;
> + else if (svn_fs_is_txn_root(root))
> + compare_root_revision = svn_fs_txn_root_base_revision(root);

Isn't it either one or the other? The old code didn't do the second
check -- do you think we shouldn't be assuming that there are only two
types of root?

Regards,
Malcolm

  • application/pgp-signature attachment: stored
Received on Fri Dec 8 19:10:04 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.