Philip Martin <philip.mar...@wandisco.com> writes: > I'm not familiar with this code, but looking at other code in the file I > tried this: > > Index: ../src-1.9/subversion/bindings/swig/include/svn_types.swg > =================================================================== > --- ../src-1.9/subversion/bindings/swig/include/svn_types.swg (revision > 1668117) > +++ ../src-1.9/subversion/bindings/swig/include/svn_types.swg (working copy) > @@ -1119,8 +1119,7 @@ > /* FIXME: This code is clearly buggy. The return value of sv_newmortal() > is immediately overwritten by the return value > of svn_swig_pl_from_md5(). */ > - ST(argvi) = sv_newmortal(); > - ST(argvi++) = svn_swig_pl_from_md5($1); > + %append_output(svn_swig_pl_from_md5($1)); > } > #endif
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. -- Philip Martin | Subversion Committer WANdisco // *Non-Stop Data*