On 10/13/2015 01:47 PM, Paul Sandoz wrote:
On 13 Oct 2015, at 13:26, Claes Redestad <claes.redes...@oracle.com> wrote:
Thus it seems fixMethodType is not necessary. I think this is confirmed when
looking at up calls to linkMethodHandleConstant and linkCallSite.
+1 - this should be a separate cleanup, though.
Yes.
The reasoning is as follows: if callerClass.getClassLoader() is sufficient for
resolving types that appear at the call site in a non-bootstrap callerClass,
then bootstrap classloader should be sufficient to resolve types that appear at
the call site in a bootstrap callerClass. Does anybody have any other opinion?
- sun.invoke.util.BytecodeDescriptor#parseMethod(java.lang.String,
java.lang.ClassLoader) is only used from the newly introduced
java.lang.invoke.MethodType#fromDescriptor
Ok, i think you have things covered, thanks,
I believe you meant "Ship it!"? :-)
Indeed!
Paul.
Ok, thanks Paul, Michael and Claes! I'll push it later today or early
tomorrow after running some more tests (nashorn for example). If anybody
else wants to comment, there's still some time (for example about the
name of the package-private method)...
Regards, Peter