On Tue, Apr 07, 2015 at 12:01:28PM +0100, Philip Martin wrote:
> Roderich Schupp <roderich.sch...@gmail.com> writes:
> 
> > On Monday, March 23, 2015 at 1:35:49 PM UTC+1, Philip Martin wrote:
> >>
> >> This makes svn_swig_pl_from_md5 follow the same pattern as
> >> svn_swig_pl_from_stream.  I've committed this to trunk as r1668618.
> 
> > Using Swig's %append_output is the correct way. I fixed another occurrence
> > of the above pattern in r1671388.
> 
> Both of these are now in the STATUS file for 1.9, 1.8 and 1.7.

I'm still experiencing the crash in 1.9.0-rc1 unless I apply the
original change I suggested to the generated
subversion/bindings/swig/perl/native/svn_delta.c.  I'm not sure how that
specific part of the file is generated, so I'll just attach a diff of
the generated file.

Cheers,
-- 
James
GPG Key: 4096R/331BA3DB 2011-12-05 James McCoy <james...@debian.org>
--- subversion-1.9.0~rc1.orig/subversion/bindings/swig/perl/native/svn_delta.c
+++ subversion-1.9.0~rc1/subversion/bindings/swig/perl/native/svn_delta.c
@@ -3845,7 +3845,6 @@
       
       
     }
-    ST(argvi) = sv_newmortal();
     {
       if (argvi >= items) EXTEND(sp,1);  ST(argvi) = svn_swig_pl_from_md5(arg3); argvi++  ;
     }

Reply via email to