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*

Reply via email to