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

David Holmes david.holmes at oracle.com
Wed Jan 24 22:29:49 UTC 2018


Thanks Karen!

David

On 25/01/2018 7:11 AM, Karen Kinnear wrote:
> 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