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