On 2017-05-10 23:05, Brian Burkhalter wrote:
Hi Roger,

Looks all right to me. I assume you will have already built the actual docs and 
clicked through the updated links. Always a bit painful …

Roger,

Did you test the actual links?

I found one needing updating:

In src/java.base/share/classes/java/io/ObjectStreamClass.java, class.html#4100 should be updated to class.html#stream-unique-identifiers.

Also, the link in src/java.base/share/classes/java/lang/String.java looks suspect. The text refers to Section 6.2 Stream Elements, but to get there the link should go to protocol.html#stream-elements. Instead it points to output.html, which is Section 2, Object Output Classes. I can't really tell which is correct, but my guess is that the text is correct and the link is not.

/Magnus


Thanks,

Brian

On May 10, 2017, at 11:22 AM, Roger Riggs <roger.ri...@oracle.com> wrote:

Please review corrections to broken javadoc links:
- links to the serialization spec now in ./specs/serialization
- links in java.lang to java/util/Spliterator
- link in ModuleLayer to Classloader
- Links using ../../../.. do not work well when they show up in some indexes; 
they should use @docRoot

webrev:
  http://cr.openjdk.java.net/~rriggs/webrev-broken-links-8180082/

Issue:
  https://bugs.openjdk.java.net/browse/JDK-8180082

Reply via email to