RFR 10: JDK-8161207: Remove extra MethodHandle subclass in MethodHandleImpl

Paul Sandoz paul.sandoz at oracle.com
Sat Jun 10 00:57:30 UTC 2017


+1

Minor thing, update the dates on the license in LambdaFormEditor and MethodHandleImpl.

Paul.

> On 9 Jun 2017, at 07:49, Ron Pressler <ron.pressler at oracle.com> wrote:
> 
> Hi.
> 
> Please review,
> Bug: https://bugs.openjdk.java.net/browse/JDK-8161207
> Webrev: http://cr.openjdk.java.net/~rpressler/jdk10/JDK-8161207-intrinsicmethodhandle/webrev/
> 
> The goal of this issue is to simplify code by removing the MethodHandleImpl.IntrinsicMethodHandle class. However, we’re not quite there yet, and the proposed change is an interim measure. The intrinsic is now stored as a field in LambdaForm.NamedFunction, but IntrinsicMethodHandle is still used. An assertion in NamedFunction’s constructor verifies that the intrinsic passed to NamedFunction matches that used in IntrinsicMethodHandle.
> 
> Ron



More information about the core-libs-dev mailing list