Re: svn_error_abort_on_malfunction doesn't

Looks about right.

--dave

On Jan 14, 2009 7:36 PM, "Julian Foad" <julianfoad_at_btopenworld.com> wrote:

David Glasser wrote: > svn_error_abort_on_malfunction claims to abort when
triggered, but it > actua...
Yes it's wrong. (At least the default implementation is. Obviously
individuals and packagers are free to replace it with an implementation
that does something different.)

"Should" be fixed before 1.6 - yes - but it's not a "must", in the sense
that it's not an API change or a guaranteed behaviour.

Is this the fix?

[[[
* subversion/libsvn_subr/error.c
 (svn_error_abort_on_malfunction): Actually abort after displaying
   the error.

--This line, and those below, will be ignored--

Index: subversion/libsvn_subr/error.c
===================================================================
--- subversion/libsvn_subr/error.c (revision 35256)
+++ subversion/libsvn_subr/error.c (working copy)
@@ -520,7 +520,8 @@ svn_error_abort_on_malfunction(svn_boole
 {
  svn_error_t *err = svn_error_raise_on_malfunction(TRUE, file, line, expr);

- svn_handle_error2(err, stderr, TRUE, "svn: ");
+ svn_handle_error2(err, stderr, FALSE, "svn: ");
+ abort();
  return err; /* Not reached. */
 }

]]]

- Julian
Received on 2009-01-15 13:51:09 CET