[Nestmates] RFR: 8195826: [Nestmates] Cleanup sun/invoke/util/VerifyAccess.java (original) (raw)

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 ALLOWNESTMATEACCESS 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