You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2013/01/07 18:41:46 UTC

svn commit: r1429917 - /subversion/trunk/subversion/libsvn_fs_fs/tree.c

Author: stefan2
Date: Mon Jan  7 17:41:46 2013
New Revision: 1429917

URL: http://svn.apache.org/viewvc?rev=1429917&view=rev
Log:
* subversion/libsvn_fs_fs/tree.c
  (svn_fs_fs__verify_root): simplify; it's not necessary to fetch the
   predecessor node as access to that revision has already been verified

Modified:
    subversion/trunk/subversion/libsvn_fs_fs/tree.c

Modified: subversion/trunk/subversion/libsvn_fs_fs/tree.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/tree.c?rev=1429917&r1=1429916&r2=1429917&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/tree.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/tree.c Mon Jan  7 17:41:46 2013
@@ -4283,8 +4283,6 @@ svn_fs_fs__verify_root(svn_fs_root_t *ro
   /* Verify explicitly the predecessor of the root. */
   {
     const svn_fs_id_t *pred_id;
-    dag_node_t *pred;
-    svn_revnum_t pred_rev;
 
     /* Only r0 should have no predecessor. */
     SVN_ERR(svn_fs_fs__dag_get_predecessor_id(&pred_id, frd->root_dir));
@@ -4300,8 +4298,7 @@ svn_fs_fs__verify_root(svn_fs_root_t *ro
     /* Check the predecessor's revision. */
     if (pred_id)
       {
-        SVN_ERR(svn_fs_fs__dag_get_node(&pred, root->fs, pred_id, pool));
-        SVN_ERR(svn_fs_fs__dag_get_revision(&pred_rev, pred, pool));
+        svn_revnum_t pred_rev = svn_fs_fs__id_rev(pred_id);
         if (pred_rev+1 != root->rev)
           /* Issue #4129. */
           return svn_error_createf(SVN_ERR_FS_CORRUPT, NULL,