JRuby needs a new release of JFFI
---------------------------------

                 Key: JRUBY-3378
                 URL: http://jira.codehaus.org/browse/JRUBY-3378
             Project: JRuby
          Issue Type: Task
            Reporter: Wayne Meissner
            Assignee: Thomas E Enebo
             Fix For: JRuby 1.2


The JFFI checked in to JRuby has a bug in it, and lacks support for a couple of 
platforms.  A later release of JFFI is needed, and the following patch needs to 
be applied to re-enable the fast path for int-only parameter functions.

{code}
0e1a0e676b116fa5993e6b7a9c503630b403fffa~1
diff --git a/src/org/jruby/ext/ffi/jffi/JFFIInvoker.java b/src/org/jruby/ext/ffi
index c964a36..dc9dbb6 100644
--- a/src/org/jruby/ext/ffi/jffi/JFFIInvoker.java
+++ b/src/org/jruby/ext/ffi/jffi/JFFIInvoker.java
@@ -98,7 +98,7 @@ public class JFFIInvoker extends org.jruby.ext.ffi.AbstractInv
     @Override
     public DynamicMethod createDynamicMethod(RubyModule module) {
         DynamicMethod dm;
-        if (false && convention == CallingConvention.DEFAULT
+        if (convention == CallingConvention.DEFAULT
             && FastIntMethodFactory.getFactory().isFastIntMethod(returnType, pa
             dm = FastIntMethodFactory.getFactory().createMethod(module,
                     function, returnType, parameterTypes);
{code}


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to