Thumbs up.
Thanks,
Serguei
On 6/19/17 13:34, Mandy Chung wrote:
+ * <a href="{@docRoot}/../specs/jpda/jpda.html”>
I sent a wrong patch. The above is the corrected link.
Mandy
On Jun 19, 2017, at 1:31 PM, serguei.spit...@oracle.com wrote:
On 6/19/17 13:23, Mandy Chung wrote:
On Jun 19, 2017, at 1:17 PM, serguei.spit...@oracle.com wrote:
Hi Mandy,
I've built the docs with your patch and the link is still not resolved.
It points to the docs/api/specs/jpda/jpda.html
It should point to docs/specs/jpda/jpda.html. I verified in my build.
Good. I'm Ok with the fix then.
Are you sure your build is correct?
No, I'm not sure.
But it is resolved if the patch is corrected with:
+ * <a href="${docRoot}/../../specs/jpda/jpda.html”>
Is it 9 or 10?
I want to fix this simple docs bug in JDK 9.
Ok, thanks.
-Serguei
Mandy
Thanks,
Serguei
On 6/19/17 12:47, Mandy Chung wrote:
Serguei,
Can you review this patch fixing the link to JPDA?
Mandy
diff --git a/src/jdk.jdi/share/classes/module-info.java
b/src/jdk.jdi/share/classes/module-info.java
--- a/src/jdk.jdi/share/classes/module-info.java
+++ b/src/jdk.jdi/share/classes/module-info.java
@@ -39,12 +39,13 @@
* creation, etc. The ability to inspect a suspended thread's state, local
* variables, stack backtrace, etc.
* <p>
- * JDI is the highest-layer of the Java Platform Debugger Architecture (JPDA).
- * For more information on the Java Platform Debugger Architecture, see the <a
- * href="{@docRoot}/../../../../technotes/guides/jpda/index.html"> Java
- * Platform Debugger Architecture documentation</a> for this release and the <a
- * href="http://java.sun.com/products/jpda";>Java Platform Debugger Architecture
- * website</a>.
+ * JDI is the highest-layer of the
+ * <a href="${docRoot}/../specs/jpda/jpda.html">
+ * Java Platform Debugger Architecture (JPDA)</a>.
+ * <p>
+ * This module includes a simple command-line debugger,
+ * <em>{@index jdb jdb tool}</em>.
+ *
* <h3>Global Exceptions</h3>
* <p>
* This section documents exceptions which apply to the entire API and are
thus
@@ -102,10 +103,6 @@
* unloaded.
* </blockquote>
*
- * <h3>jdb</h3>
- *
- * <em>{@index jdb jdb tool}</em> is a simple command-line debugger provided
- * in this module.
*
* <dl style="font-family:'DejaVu Sans', Arial, Helvetica, sans serif">
* <dt class="simpleTagLabel">Tool Guides:
@@ -119,6 +116,8 @@
*
* @moduleGraph
* @since 9
+ * @see <a href="${docRoot}/../specs/jpda/jpda.html">
+ * Java Platform Debugger Architecture (JPDA)</a>
*/
module jdk.jdi {
requires jdk.attach;
