RFR: JDK-8300357: Use generalized see and link tags in java.management [v3]
Joe Darcy
darcy at openjdk.org
Wed Jan 18 19:13:44 UTC 2023
On Wed, 18 Jan 2023 12:22:01 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Joe Darcy has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Improve link target per code review comments.
>
> src/java.management/share/classes/javax/management/remote/JMXConnectorFactory.java line 140:
>
>> 138: * implementation may choose to find providers by other means. For
>> 139: * example, it may support {@linkplain
>> 140: * java.base/java.util.ServiceLoader##developing-service-providers service providers},
>
> is `java.base/` needed here? I am a bit surprised - it's first time I see a module name used in a regular link target.
Updated.
> src/java.management/share/classes/javax/management/remote/JMXConnectorServerFactory.java line 129:
>
>> 127: * <code>MalformedURLException</code> if there is none. An
>> 128: * implementation may choose to find providers by other means. For
>> 129: * example, it may support {@linkplain
>
> Same remark regarding `java.base/` prefix.
For the connector server factory, "java.util" is still needed in the link since I didn't want to change the import statements of the class.
-------------
PR: https://git.openjdk.org/jdk/pull/12058
More information about the serviceability-dev
mailing list