Re: Missing editor->abort_edit() in wc_to_repos_copy() ?

Gerco Ballintijn wrote:
> Hi,
>
> I noticed the following. svn_client__do_commit() is called by both
> svn_client_commit4() and wc_to_repos_copy(). svn_client__do_commit()
> ends with editor->close_edit() on success, but no editor->abort_edit()
> on failure. Now, svn_client_commit4() does call editor->abort_edit()
> when svn_client__do_commit() returns unsuccessfully, but as far as I
> can tell wc_to_repos_copy() does not call editor->abort_edit() when
> svn_client__do_commit(). Isn't this a bug?

I would say, "yes, bug." (And I've just independently confirmed that my
reading of the code matches yours, having originally seen only the subject
line of your post.)

-- 
C. Michael Pilato <cmpilato@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand