Daniel Rall wrote:
>Greg Stein <gstein@lyra.org> writes:
>
>
>>On Tue, Feb 11, 2003 at 05:57:46PM -0600, dlr@tigris.org wrote:
>>
>>
>>>...
>>>+++ trunk/subversion/bindings/swig/apr.i Tue Feb 11 17:57:29 2003
>>>...
>>> /* -----------------------------------------------------------------------
>>> handle apr_file_t *
>>> */
>>>@@ -153,7 +162,7 @@
>>> apr_file_t ** is always an OUT param
>>> */
>>>
>>>-%typemap(in,numinputs=0) apr_file_t ** (apr_file_t *temp)
>>>+%typemap(python, in, numinputs=0) apr_file_t ** (apr_file_t *temp)
>>> "$1 = &temp;";
>>>
>>>
>>I see nothing Python specific there. Why was this change made?
>>
>>
>
>Well, we were on a roll with breaking the build and just didn't see a
>need to stop ... fixed this one in r4866. There may be others.
>
>
It is Python specific since it uses "numinputs=0", thus relying on
t_output_helper to return the result in a tuple.
In Java, the binding must/should work by returning the output variable
in a in/out parameter, which is done by using an array of
SWIGTYPE_p_apr_file_t of length one - SWIG does this automatically, but
the typemap above overrides (incorrectly).
I'm sorry if my patches otherwise broke the build - I hadn't imagined
that it could hurt the Python binding to add the qualifier.
I'd better install Python here and test with it in the future.
-Jesper
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Feb 13 10:22:42 2003