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.
   */
-@Deprecated
+@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.
   */
-@Deprecated
+@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.
   */
-@Deprecated
+@Deprecated(since="1.2", forRemoval=true)
  public abstract class Signer extends Identity {

      private static final long serialVersionUID = -1763464102261361480L;

Reply via email to