Re: JDK 9 RFR of JDK-8069277: Doclint regression in java/security/KeyStore.java

[Joseph D. Darcy]

Hi Max,

Yes; both should be "linkplain" -- I'll push with that fix.

Thanks,

-Joe

On 2/9/2015 4:54 PM, Wang Weijun wrote:

Should the "providers" link in the 2nd method also be "linkplain"?

--Max

On Feb 10, 2015, at 08:28, Joseph D. Darcy <joe.da...@oracle.com> wrote:

Hello,

As the final purge of doclint warnings from the java.base module approaches, 
please review the changes for

    JDK-8069277: Doclint regression in java/security/KeyStore.java
    http://cr.openjdk.java.net/~darcy/8069277.0/

Patch below.

Thanks,

-Joe

--- old/src/java.base/share/classes/java/security/KeyStore.java 2015-02-09 
16:25:01.853908872 -0800
+++ new/src/java.base/share/classes/java/security/KeyStore.java 2015-02-09 
16:25:01.529912945 -0800
@@ -1618,11 +1618,13 @@
      * integrity check.
      *
      * <p>
-     * This method traverses the list of registered security {@link Providers},
-     * starting with the most preferred Provider.
-     * For each {@link KeyStoreSpi} implementation supported by a Provider,
-     * it invokes the {@link engineProbe} method to determine if it supports
-     * the specified keystore.
+     * This method traverses the list of registered security
+     * {@linkplain Provider providers}, starting with the most
+     * preferred Provider.
+     * For each {@link KeyStoreSpi} implementation supported by a
+     * Provider, it invokes the {@link
+     * KeyStoreSpi#engineProbe(InputStream) engineProbe} method to
+     * determine if it supports the specified keystore.
      * A new KeyStore object is returned that encapsulates the KeyStoreSpi
      * implementation from the first Provider that supports the specified file.
      *
@@ -1672,11 +1674,12 @@
      * unlock the keystore data or perform an integrity check.
      *
      * <p>
-     * This method traverses the list of registered security {@link Providers},
-     * starting with the most preferred Provider.
-     * For each {@link KeyStoreSpi} implementation supported by a Provider,
-     * it invokes the {@link engineProbe} method to determine if it supports
-     * the specified keystore.
+     * This method traverses the list of registered security {@link
+     * Provider providers}, starting with the most preferred Provider.
+     * For each {@link KeyStoreSpi} implementation supported by a
+     * Provider, it invokes the {@link
+     * KeyStoreSpi#engineProbe(InputStream) engineProbe} method to
+     * determine if it supports the specified keystore.
      * A new KeyStore object is returned that encapsulates the KeyStoreSpi
      * implementation from the first Provider that supports the specified file.
      *