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++ ; }