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

Reply via email to