On Sat, May 5, 2012 at 5:35 PM, <hwri...@apache.org> wrote:
>...
> +++ subversion/branches/ev2-export/subversion/libsvn_client/commit_util.c Sat
> May 5 21:35:28 2012
> @@ -1564,7 +1564,8 @@ do_item_commit(svn_client_commit_item3_t
> /* If this item is supposed to be deleted, do so. */
> if (item->state_flags & SVN_CLIENT_COMMIT_ITEM_DELETE)
> {
> - err = svn_editor_delete(editor, repos_relpath, item->revision);
> + err = svn_error_trace(
> + svn_editor_delete(editor, repos_relpath, item->revision));
>
> if (err)
> goto fixup_error;
It seems that you could move the call to fixup_commit_error() into the
one caller of do_item_commit(), and then convert all of these blocks
to a normal SVN_ERR() (which has proper tracing).
The goto's were needed when do_item_commit() was directly called by
the path driver, iirc.
Cheers,
-g
