RFR(xs): 8156810 remove redundant sentence in SecurityManager.checkMemberAccess doc

Stuart Marks stuart.marks at oracle.com
Wed May 11 23:47:23 UTC 2016


Hi all,

Regarding this bug,

     https://bugs.openjdk.java.net/browse/JDK-8156810

I had recently "upgraded" the deprecation annotation of 
SecurityManager.checkMemberAccess() to include forRemoval=true. [1] [2] This 
included the addition of some text about removal in a future version. 
Unfortunately the deprecation text now reads, in part,

     This method will be changed in a future release to check the
     permission java.security.AllPermission. This method is subject
     to removal in a future version of Java SE.

Obviously the first sentence is irrelevant now that we're going to remove the 
method entirely.

Please review diff appended below that removes the first sentence.

Thanks,

s'marks



[1] https://bugs.openjdk.java.net/browse/JDK-8145468

[2] http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-April/040192.html



diff -r 5f4aff1d2242 src/java.base/share/classes/java/lang/SecurityManager.java
--- a/src/java.base/share/classes/java/lang/SecurityManager.java	Mon May 09 
18:11:02 2016 +0800
+++ b/src/java.base/share/classes/java/lang/SecurityManager.java	Wed May 11 
16:46:11 2016 -0700
@@ -1627,8 +1627,7 @@
       * @deprecated This method relies on the caller being at a stack depth
       *             of 4 which is error-prone and cannot be enforced by the 
runtime.
       *             Users of this method should instead invoke {@link 
#checkPermission}
-     *             directly.  This method will be changed in a future release
-     *             to check the permission {@code java.security.AllPermission}.
+     *             directly.
       *             This method is subject to removal in a future version of 
Java SE.
       *
       * @see java.lang.reflect.Member



More information about the core-libs-dev mailing list