You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by hw...@apache.org on 2012/02/24 02:07:46 UTC

svn commit: r1293051 - /subversion/trunk/subversion/libsvn_repos/dump.c

Author: hwright
Date: Fri Feb 24 01:07:46 2012
New Revision: 1293051

URL: http://svn.apache.org/viewvc?rev=1293051&view=rev
Log:
When doing repos verification, be sure we actually close the verification editor.

Current number of Ev2 test failures: 6

* subversion/libsvn_repos/dump.c
  (svn_repos_verify_fs2): Close the verifying dump editor when done with it.

Modified:
    subversion/trunk/subversion/libsvn_repos/dump.c

Modified: subversion/trunk/subversion/libsvn_repos/dump.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_repos/dump.c?rev=1293051&r1=1293050&r2=1293051&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_repos/dump.c (original)
+++ subversion/trunk/subversion/libsvn_repos/dump.c Fri Feb 24 01:07:46 2012
@@ -1432,6 +1432,10 @@ svn_repos_verify_fs2(svn_repos_t *repos,
       SVN_ERR(svn_repos_replay2(to_root, "", SVN_INVALID_REVNUM, FALSE,
                                 cancel_editor, cancel_edit_baton,
                                 NULL, NULL, iterpool));
+      /* While our editor close_edit implementation is a no-op, we still
+         do this for completeness. */
+      SVN_ERR(cancel_editor->close_edit(cancel_edit_baton, iterpool));
+
       SVN_ERR(svn_fs_revision_proplist(&props, fs, rev, iterpool));
 
       if (notify_func)