Looks fine.

Mandy

On 7/11/18 5:01 PM, Paul Sandoz wrote:
Hi,

Please review this fix for 11:

http://cr.openjdk.java.net/~psandoz/jdk/JDK-8207027-lookup-access-class-array-type/webrev/

 The fix removes an old and now incorrect assert statement. This
statement causes MethodHandles.Lookup.accessClass to fail when system
assertions are enabled and an array class in the same package as the
lookup class is passed as an argument.

Usages of MethodHandles.Lookup.accessClass such as by
ConstantBootstraps.arrayVarHandle will also fail, which results in a
JCK test failure (reported in bug).

Thanks, Paul.

Reply via email to