Looks good.

--Sean

On 5/11/16 4:47 PM, Stuart Marks wrote:
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

Reply via email to