Re: Missing editor->abort_edit() in wc_to_repos_copy() ?
Gerco Ballintijn wrote:
> 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 <firstname.lastname@example.org>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on Mon Mar 12 16:34:32 2007
This is an archived mail posted to the Subversion Dev