Re: [11] RFR 8207027 Lookup.accessClass fails for an array type in the same package when assertions are enabled

2018-07-12 Thread mandy chung
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

Re: [11] RFR 8207027 Lookup.accessClass fails for an array type in the same package when assertions are enabled

2018-07-12 Thread Claes Redestad
+1 /Claes On 2018-07-12 02:01, 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

[11] RFR 8207027 Lookup.accessClass fails for an array type in the same package when assertions are enabled

2018-07-11 Thread Paul Sandoz
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