[Swig-devel] [ swig-Bugs-3291036 ] ruby 1.9 rb_respond_to changes break SWIG generated code

[SourceForge.net <no...@so...>]

Bugs item #3291036, was opened at 2011-04-21 12:15
Message generated for change (Comment added) made by brettwilliams
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=101645&aid=3291036&group_id=1645

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: ruby
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Brett (brettwilliams)
Assigned to: Nobody/Anonymous (nobody)
Summary: ruby 1.9 rb_respond_to changes break SWIG generated code

Initial Comment:
SWIG generates some code that compares the result of rb_respond_to to Qtrue.  This causes runtime errors in ruby 1.9.  The fix I think should be to remove the "== Qtrue", since both 1.8 and 1.9 should give you the correct result.  In neither version does it appear to me that it is safe to assume you are getting a Qtrue/Qfalse result.  I've not unravelled it far enough to see why this was working in previous versions.

It may be that the original SWIG implementation saw the code of obj_respond_to (a static function not accessible to swig) that does return Qtrue or Qfalse.  That one is the function ultimately called by the interpreter when you call the ruby method respond_to?, but I don't think it is safe to treat rb_respond_to as returning Qtrue for a "true" result in either version.

See rb_respond_to, rb_obj_respond_to in the ruby source code (vm_method.c for 1.9, eval.c for 1.8).

Examples of where SWIG does this include these two functions.  These functions aren't used anywhere except in my own custom typemaps (at least in my test case) but they will not work.  At least in my test case SWIG doesn't do this anywhere else.
SWIGINTERN
int SWIG_Ruby_isCallable( VALUE proc )
{
  if ( rb_respond_to( proc, swig_call_id ) == Qtrue )
    return 1;
  return 0;
}

and:

SWIGINTERN
int SWIG_Ruby_arity( VALUE proc, int minimal )
{
  if ( rb_respond_to( proc, swig_arity_id ) == Qtrue )
    {
      VALUE num = rb_funcall( proc, swig_arity_id, 0 );
      int arity = NUM2INT(num);
      if ( arity < 0 && (arity+1) < -minimal ) return 1;
      if ( arity == minimal ) return 1;
      return 1;
    }
  return 0;
}



----------------------------------------------------------------------

>Comment By: Brett (brettwilliams)
Date: 2011-05-24 13:03

Message:
I have not attempted that.  I'll try to figure that out.

----------------------------------------------------------------------

Comment By: Olly Betts (olly)
Date: 2011-05-23 20:59

Message:
Did you try running the testsuite with this change?

make check-ruby-test-suite

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=101645&aid=3291036&group_id=1645