Looks good. --Sean
On 10/27/17 2:54 AM, Weijun Wang wrote:
Turns out there is a javac warning for "removal". Please review again at http://cr.openjdk.java.net/~weijun/8159535/webrev.00 Thanks MaxOn Oct 20, 2017, at 11:56 AM, Weijun Wang <weijun.w...@oracle.com> wrote: Please review this patch. A CSR [1] is also filed. diff --git a/src/java.base/share/classes/javax/security/auth/Policy.java b/src/java.base/share/classes/javax/security/auth/Policy.java --- a/src/java.base/share/classes/javax/security/auth/Policy.java +++ b/src/java.base/share/classes/javax/security/auth/Policy.java @@ -152,11 +152,12 @@ * * These two APIs provide callers the means to query the * Policy for Principal-based Permission entries. + * This class is subject to removal in a future version of Java SE. * * @since 1.4 * @see java.security.Security security properties */ -@Deprecated(since="1.4") +@Deprecated(since="1.4", forRemoval=true) public abstract class Policy { private static Policy policy; Thanks Max [1] https://bugs.openjdk.java.net/browse/JDK-8189621
