[Nestmates] RFR: 8195826: [Nestmates] Cleanup sun/invoke/util/VerifyAccess.java

Karen Kinnear karen.kinnear at oracle.com
Wed Jan 24 21:11:01 UTC 2018


David,

Actually thank you for the response and the tests.
The code looks good.

thanks,
Karen


> On Jan 22, 2018, at 5:33 PM, David Holmes <david.holmes at oracle.com> wrote:
> 
> On 23/01/2018 8:27 AM, Karen Kinnear wrote:
>> Forgot to ask - do you have a test for this?
> 
> Numerous MethodHandle tests exercise this access checking logic:
> 
> runtime/Nestmates/privateConstructors/TestMethodHandles.java
> runtime/Nestmates/privateFields/TestMethodHandles.java
> runtime/Nestmates/privateMethods/TestMethodHandles.java
> runtime/Nestmates/privateStaticFields/TestMethodHandles.java
> runtime/Nestmates/privateStaticMethods/TestMethodHandles.java
> 
> Thanks,
> David
> 
>> thanks,
>> Karen
>>> On Jan 22, 2018, at 1:31 AM, David Holmes <david.holmes at oracle.com> wrote:
>>> 
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8195826
>>> webrev: http://cr.openjdk.java.net/~dholmes/8195826/webrev
>>> 
>>> Get rid of old experimental ALLOW_NESTMATE_ACCESS support and recast directly using Reflection.areNestMates(a,b).
>>> 
>>> Update quote of JVMS 5.4.4 access rules.
>>> 
>>> Thanks,
>>> David



More information about the valhalla-dev mailing list