RFR 8159535: Mark deprecated javax.security.auth.Policy API with forRemoval=true

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR 8159535: Mark deprecated javax.security.auth.Policy API with forRemoval=true

Weijun Wang
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
Reply | Threaded
Open this post in threaded view
|

Re: RFR 8159535: Mark deprecated javax.security.auth.Policy API with forRemoval=true

Sean Mullan
Looks good to me. Please add a release note subtask to the issue.

--Sean

On 10/19/17 11:56 PM, Weijun Wang 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
>
Reply | Threaded
Open this post in threaded view
|

Re: RFR 8159535: Mark deprecated javax.security.auth.Policy API with forRemoval=true

Weijun Wang
https://bugs.openjdk.java.net/browse/JDK-8189746 created.

--Max

> On Oct 20, 2017, at 10:29 PM, Sean Mullan <[hidden email]> wrote:
>
> Looks good to me. Please add a release note subtask to the issue.
>
> --Sean
>
> On 10/19/17 11:56 PM, Weijun Wang 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

Reply | Threaded
Open this post in threaded view
|

Re: RFR 8159535: Mark deprecated javax.security.auth.Policy API with forRemoval=true

Weijun Wang
In reply to this post by Weijun Wang
Turns out there is a javac warning for "removal". Please review again at

   http://cr.openjdk.java.net/~weijun/8159535/webrev.00

Thanks
Max

> On Oct 20, 2017, at 11:56 AM, Weijun Wang <[hidden email]> 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

Reply | Threaded
Open this post in threaded view
|

Re: RFR 8159535: Mark deprecated javax.security.auth.Policy API with forRemoval=true

Sean Mullan
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
> Max
>
>> On Oct 20, 2017, at 11:56 AM, Weijun Wang <[hidden email]> 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
>