On 11/14/19 8:04 AM, Mandy Chung wrote:


On 11/14/19 2:33 AM, Alan Bateman wrote:
On 14/11/2019 04:57, Mandy Chung wrote:

Updated in place:

http://cr.openjdk.java.net/~mchung/jdk14/8233527/webrev.02/
http://cr.openjdk.java.net/~mchung/jdk14/8233527/specdiff/
The addition of hasFullPrivilegeAccess looks okay and probably the right thing to do.

For the @deprecated message on the old method it might be simpler to say that the method was originally designed to test PRIVATE access but has since changed to test full privilege access.

It was designed to test full privilege access while private access implies full privilege access as a Lookup with private access includes module access.


I updated to:

This method was originally designed to test PRIVATE access that implies full privilege access but PRIVATE access has since become independent of PRIVATE access. It is recommended to call hasFullPrivilegeAccess() instead.

http://cr.openjdk.java.net/~mchung/jdk14//8233527/specdiff/java.base/java/lang/invoke/MethodHandles.Lookup.html

Mandy

Reply via email to