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

Sean Mullan sean.mullan at oracle.com
Thu May 12 00:33:41 UTC 2016


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



More information about the core-libs-dev mailing list