RFR 8157730: Mark deprecated java.security.{Identity, IdentityScope, Signer} APIs with forRemoval=true

[Vincent Ryan]

Sure. (It works for packages too.)

> On 17 Jun 2016, at 18:54, ecki at zusammenkunft.net wrote:
> 
> Hello,
> 
> While you are at that code, can the JavaDoc  @code classnames be changed into @link as well? (Not sure if it works for packages).
> 
> Gruss
> Bernd
> 
> -- 
> http://bernd.eckenfels.net
> 
> -----Original Message-----
> From: Sean Mullan <sean.mullan at oracle.com>
> To: Vincent Ryan <vincent.x.ryan at oracle.com>, OpenJDK <security-dev at openjdk.java.net>
> Sent: Fr., 17 Juni 2016 16:41
> Subject: Re: RFR 8157730: Mark deprecated java.security.{Identity, IdentityScope, Signer} APIs with forRemoval=true
> 
> Looks fine to me.
> 
> --Sean
> 
> On 06/17/2016 08:52 AM, Vincent Ryan wrote:
>> Three Identity-related classes were deprecated in JDK 1.2.
>> Please review the patch below that marks them as candidates for removal
>> in a future JDK release.
>> 
>> See http://openjdk.java.net/jeps/277 for details of the enhanced
>> deprecation annotation.
>> 
>> Thanks.
>> 
>> 
>> diff --git a/src/java.base/share/classes/java/security/Identity.java
>> b/src/java.base/share/classes/java/security/Identity.java
>> --- a/src/java.base/share/classes/java/security/Identity.java
>> +++ b/src/java.base/share/classes/java/security/Identity.java
>> @@ -55,8 +55,9 @@
>>   * replaced by {@code java.security.KeyStore}, the
>>   * {@code java.security.cert} package, and
>>   * {@code java.security.Principal}.
>> + * This class is subject to removal in a future version of Java SE.
>>   */
>> - at Deprecated
>> + at Deprecated(since="1.2", forRemoval=true)
>>  public abstract class Identity implements Principal, Serializable {
>> 
>>      /** use serialVersionUID from JDK 1.1.x for interoperability */
>> 
>> 
>> diff --git
>> a/src/java.base/share/classes/java/security/IdentityScope.java
>> b/src/java.base/share/classes/java/security/IdentityScope.java
>> --- a/src/java.base/share/classes/java/security/IdentityScope.java
>> +++ b/src/java.base/share/classes/java/security/IdentityScope.java
>> @@ -60,8 +60,12 @@
>>   * replaced by {@code java.security.KeyStore}, the
>>   * {@code java.security.cert} package, and
>>   * {@code java.security.Principal}.
>> + * This class is subject to removal in a future version of Java SE.
>> + *
>> + * Note that the security property {@code policy.ignoreIdentityScope}
>> + * is only applicable to these APIs and is also a candidate for removal.
>>   */
>> - at Deprecated
>> + at Deprecated(since="1.2", forRemoval=true)
>>  public abstract
>>  class IdentityScope extends Identity {
>> 
>> 
>> diff --git a/src/java.base/share/classes/java/security/Signer.java
>> b/src/java.base/share/classes/java/security/Signer.java
>> --- a/src/java.base/share/classes/java/security/Signer.java
>> +++ b/src/java.base/share/classes/java/security/Signer.java
>> @@ -43,8 +43,9 @@
>>   * replaced by {@code java.security.KeyStore}, the
>>   * {@code java.security.cert} package, and
>>   * {@code java.security.Principal}.
>> + * This class is subject to removal in a future version of Java SE.
>>   */
>> - at Deprecated
>> + at Deprecated(since="1.2", forRemoval=true)
>>  public abstract class Signer extends Identity {
>> 
>>      private static final long serialVersionUID = -1763464102261361480L;
>>