In jnimeth.c, it looks like line 73 should say result.unionselector, or
else the return value of the function should be jvalue.  I will not be
modifying jnimeth.c at all for Classpath integration, so the fix should go
in the main tree.  I don't know if this would cause any problems at all, but
better safe than sorry.
        Also, if you're interested, in libnative/java.lang.reflect, there is a
serious error in invokeNative().  It does not handle void methods or methods
that return primitive types correctly, instead simply using CallObjectMethod
in all cases.  I assume you can see how easily this will cause segfaults.
This will be fixed in the Japhar+Classpath stuff, though, so unless you are
desperate to get java.lang.reflect for Sun working right now, you may as
well wait and copy stuff from my patch when it's done.
--John Keiser

Reply via email to