Please review the patch below. I don't know if all those jdk.crypto.* modules will be listed in the doc, but it's nice to include a comment.

Thanks
Max

diff --git a/src/jdk.policytool/share/classes/module-info.java b/src/jdk.policytool/share/classes/module-info.java
--- a/src/jdk.policytool/share/classes/module-info.java
+++ b/src/jdk.policytool/share/classes/module-info.java
@@ -23,6 +23,14 @@
  * questions.
  */

+/**
+ * GUI tool for managing policy files.
+ *
+ * @since 9
+ * @deprecated The policytool tool has been deprecated and
+ * is planned to be removed in a future release.
+ */
+@Deprecated
 module jdk.policytool {
     requires java.desktop;
     requires java.logging;

diff --git a/src/jdk.crypto.cryptoki/share/classes/module-info.java b/src/jdk.crypto.cryptoki/share/classes/module-info.java
--- a/src/jdk.crypto.cryptoki/share/classes/module-info.java
+++ b/src/jdk.crypto.cryptoki/share/classes/module-info.java
@@ -23,6 +23,11 @@
  * questions.
  */

+/**
+ * The SunPKCS11 security provider.
+ *
+ * @since 9
+ */
 module jdk.crypto.cryptoki {
     // Depends on SunEC provider for EC related functionality
     requires jdk.crypto.ec;
diff --git a/src/jdk.crypto.ec/share/classes/module-info.java b/src/jdk.crypto.ec/share/classes/module-info.java
--- a/src/jdk.crypto.ec/share/classes/module-info.java
+++ b/src/jdk.crypto.ec/share/classes/module-info.java
@@ -23,6 +23,11 @@
  * questions.
  */

+/**
+ * The SunEC security provider.
+ *
+ * @since 9
+ */
 module jdk.crypto.ec {
     provides java.security.Provider with sun.security.ec.SunEC;
 }
diff --git a/src/jdk.crypto.mscapi/windows/classes/module-info.java b/src/jdk.crypto.mscapi/windows/classes/module-info.java
--- a/src/jdk.crypto.mscapi/windows/classes/module-info.java
+++ b/src/jdk.crypto.mscapi/windows/classes/module-info.java
@@ -23,6 +23,11 @@
  * questions.
  */

+/**
+ * The SunMSCAPI security provider.
+ *
+ * @since 9
+ */
 module jdk.crypto.mscapi {
     provides java.security.Provider with sun.security.mscapi.SunMSCAPI;
 }
diff --git a/src/jdk.crypto.ucrypto/solaris/classes/module-info.java b/src/jdk.crypto.ucrypto/solaris/classes/module-info.java
--- a/src/jdk.crypto.ucrypto/solaris/classes/module-info.java
+++ b/src/jdk.crypto.ucrypto/solaris/classes/module-info.java
@@ -23,6 +23,11 @@
  * questions.
  */

+/**
+ * The OracleUCrypto security provider.
+ *
+ * @since 9
+ */
 module jdk.crypto.ucrypto {
provides java.security.Provider with com.oracle.security.ucrypto.UcryptoProvider;
 }

Thanks
Max

Reply via email to