RFR: 8293333: Broken links in JDI specification

Chris Plummer cjplummer at openjdk.org
Wed Sep 7 06:34:47 UTC 2022


On Wed, 7 Sep 2022 05:51:53 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:

> The JDI files `com/sun/jdi/ClassLoaderReference.java` and `com/sun/jdi/VirtualMachine.java` have incorrect links to the JVM TI spec with the extra folder name `jvmti`. 
> 
> The fix is to get rid unneeded part from the links:
> 
> --- a/src/jdk.jdi/share/classes/com/sun/jdi/ClassLoaderReference.java
> +++ b/src/jdk.jdi/share/classes/com/sun/jdi/ClassLoaderReference.java
> @@ -93,7 +93,7 @@ public interface ClassLoaderReference extends ObjectReference {
>       * classes which this class loader can find by name. The list
>       * has length 0 if no classes are visible to this classloader.
>       *
> -     * @see <a href="{@docRoot}/../specs/jvmti/jvmti.html#GetClassLoaderClasses">
> +     * @see <a href="{@docRoot}/../specs/jvmti.html#GetClassLoaderClasses">
>       *     JVM TI GetClassLoaderClasses</a>
>       */
>      List<ReferenceType> visibleClasses();
> diff --git a/src/jdk.jdi/share/classes/com/sun/jdi/VirtualMachine.java b/src/jdk.jdi/share/classes/com/sun/jdi/VirtualMachine.java
> index 12cdd566989..64c91c8d1bb 100644
> --- a/src/jdk.jdi/share/classes/com/sun/jdi/VirtualMachine.java
> +++ b/src/jdk.jdi/share/classes/com/sun/jdi/VirtualMachine.java
> @@ -144,7 +144,7 @@ public interface VirtualMachine extends Mirror {
>       *
>       * @return a list of {@link ReferenceType} objects, each mirroring
>       * a loaded type in the target VM.
> -     * @see <a href="{@docRoot}/../specs/jvmti/jvmti.html#GetLoadedClasses">
> +     * @see <a href="{@docRoot}/../specs/jvmti.html#GetLoadedClasses">
>       * JVM TI GetLoadedClasses</a> regarding how class and interface creation can be triggered
>       */
>      List<ReferenceType> allClasses();

Marked as reviewed by cjplummer (Reviewer).

-------------

PR: https://git.openjdk.org/jdk/pull/10191


More information about the serviceability-dev mailing list