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.Darcy at 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.
>>       *
>>