You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ju...@apache.org on 2014/03/06 20:01:50 UTC
svn commit: r1575007 - /subversion/trunk/subversion/libsvn_repos/dump.c
Author: julianfoad
Date: Thu Mar 6 19:01:49 2014
New Revision: 1575007
URL: http://svn.apache.org/r1575007
Log:
* subversion/libsvn_repos/dump.c
(svn_repos_verify_fs3): Add a comment. Remove a misplaced
comment. Tweak white space.
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=1575007&r1=1575006&r2=1575007&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_repos/dump.c (original)
+++ subversion/trunk/subversion/libsvn_repos/dump.c Thu Mar 6 19:01:49 2014
@@ -2216,8 +2216,8 @@ svn_repos_verify_fs3(svn_repos_t *repos,
forwarding structure for notifications from inside svn_fs_verify(). */
if (notify_func)
{
- notify = svn_repos_notify_create(svn_repos_notify_verify_rev_end,
- pool);
+ notify = svn_repos_notify_create(svn_repos_notify_verify_rev_end, pool);
+
verify_notify = verify_fs2_notify_func;
verify_notify_baton = apr_palloc(pool, sizeof(*verify_notify_baton));
verify_notify_baton->notify_func = notify_func;
@@ -2243,6 +2243,8 @@ svn_repos_verify_fs3(svn_repos_t *repos,
svn_error_clear(err);
if (!keep_going)
+ /* ### Jump to "We're done" and so send the final notification,
+ for consistency? */
return svn_error_createf(SVN_ERR_REPOS_CORRUPTED, NULL,
_("Repository '%s' failed to verify"),
svn_dirent_local_style(svn_repos_path(repos,
@@ -2290,7 +2292,6 @@ svn_repos_verify_fs3(svn_repos_t *repos,
notify_func(notify_baton, notify, iterpool);
}
- /* Per-backend verification. */
svn_pool_destroy(iterpool);
if (found_corruption)