+1 Minor thing, update the dates on the license in LambdaFormEditor and MethodHandleImpl.
Paul. > On 9 Jun 2017, at 07:49, Ron Pressler <ron.press...@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
