Re: svn commit: r36171 - in trunk/subversion/bindings/swig: include ruby/svn ruby/test

["Hyrum K. Wright" <hy...@mail.utexas.edu>]

On Feb 27, 2009, at 8:10 PM, Joe Swatosh wrote:

> On Fri, Feb 27, 2009 at 6:58 AM, Hyrum K. Wright
> <hy...@mail.utexas.edu> wrote:
>> On Feb 27, 2009, at 12:05 AM, Joe Swatosh wrote:
>>
>>> Author: joeswatosh
>>> Date: Thu Feb 26 22:05:28 2009
>>> New Revision: 36171
>>>
>>> Log:
>>> Fix failing Ruby bindings test in the spirit of the fix for Python
>>> in r21423:
>>> "Fix segfault in Python tests by deleting the broken argout typemap
>>> for the
>>> 'result_digest' parameter in svn_txdelta_apply. Instead, ignore the
>>> parameter
>>> altogether."
>>>
>>> Updated to follow djames recommendation to not create an unused
>>> temporary.
>>>
>>> * subversion/bindings/swig/include/svn_types.swg
>>> (result_digest): Remove typemap for Ruby. Instead, ignore
>>>  the parameter.
>>> * subversion/bindings/swig/ruby/svn/delta.rb
>>> (Svn::Delta.apply): Don't expect a digest to be returned by
>>>  Svn::Delta.txdelta_apply_wrapper and don't return it.
>>> * subversion/bindings/swig/ruby/test/test_delta.rb
>>> (SvnDeltaTest#test_apply): Don't expect a digest returned with the
>>> handler
>>>  from Svn::Delta.apply.
>>>
>>> Reviewed by: djames
>>>             hwright
>>
>> Heh.  I don't recall actually reviewing this, just saying something
>> like "if you think it improves the state of the world, it can't  
>> hurt."
>>
>
> Sorry I misinterpreted you.
>
> Has it improved the state of the world? :-)

I think that it has.  There is now only one swig-rb test currently  
failing on trunk.

-Hyrum

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1268454