On Fri, 19 Jul 2024 11:11:38 GMT, Nizar Benalla <nbena...@openjdk.org> wrote:
> Can I get a review for this change that fixes some broken links in javadoc > comments? The new docs are hosted > [here](https://cr.openjdk.org/~nbenalla/GeneratedDocs/8336039-warnings-links/). > > It's mostly fixing some relative links. > If using `{@docroot}` isn't ideal I can change it. > > Here is the result of running `diff -r docs-master docs` on the docs from > master vs and after these changes > > > diff -r > docs-master/api/java.base/java/lang/classfile/components/CodeStackTracker.html > docs/api/java.base/java/lang/classfile/components/CodeStackTracker.html > 106c106 > < <p> > --- >> </p> > diff -r docs-master/api/java.base/java/lang/classfile/package-summary.html > docs/api/java.base/java/lang/classfile/package-summary.html > 99c99 > < <p> > --- >> > 106c106 > < <p> > --- >> > 618c618 > < <p> > --- >> > 755c755 > < <p> > --- >> > 783c783 > < <p> > --- >> > diff -r docs-master/api/java.base/java/lang/foreign/Arena.html > docs/api/java.base/java/lang/foreign/Arena.html > 142c142 > < the segments allocated by it) becomes <a > href="../../../java/lang/ref/package.html#reachability">unreachable</a>, > --- >> the segments allocated by it) becomes <a >> href="../../../java/lang/ref/package-summary.html#reachability">unreachable</a>, > diff -r docs-master/api/java.base/java/lang/foreign/MemorySegment.Scope.html > docs/api/java.base/java/lang/foreign/MemorySegment.Scope.html > 120c120 > < as long as it is <a > href="../../../java/lang/ref/package.html#reachability">reachable</a>. > --- >> as long as it is <a >> href="../../../java/lang/ref/package-summary.html#reachability">reachable</a>. > diff -r docs-master/api/java.base/java/lang/foreign/MemorySegment.html > docs/api/java.base/java/lang/foreign/MemorySegment.html > 1420c1420 > < kept <a > href="../../../java/lang/ref/package.html#reachability">reachable</a> > --- >> kept <a >> href="../../../java/lang/ref/package-summary.html#reachability">reachable</a> > 1833c1833 > < <a > href="../../../java/lang/ref/package.html#reachability">unreachable</a>. > --- >> <a >> href="../../../java/lang/ref/package-summary.html#reachability">unreachable</a>. > 1899c1899 > < <a > href="../../../java/lang/ref/package.html#reachability">unreachable</a>. > --- >> <a >> href="../../../java/lang/ref/package-summary.html#reachability">unreachable</a>. > diff -r docs-master/api/java.base/java/lang/foreign/SymbolLookup.html > docs/api/java.base/java/lang/foreign/SymbolLookup.html > 395c395 > < <a > href="../../../java/lang/ref/package.html#reachability">unreachable</a>. The > --- >> <a href="../../../j... src/java.base/share/classes/java/util/concurrent/StructuredTaskScope.java line 1013: > 1011: * <p> Construction captures the current thread's {@linkplain > ScopedValue scoped > 1012: * value} bindings for inheritance by threads started in the > task scope. The > 1013: * <a > href="{@docRoot}/java.base/java/util/concurrent/StructuredTaskScope.html#TreeStructure">Tree > Structure</a> section in the class description (untested, but should work) Suggestion: * {@linkplain StructuredTaskScope##TreeStructure Tree Structure} section in the class description ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/20251#discussion_r1684324692
