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...

This pull request has now been integrated.

Changeset: 92de2b2d
Author:    Nizar Benalla <nbena...@openjdk.org>
Committer: Chen Liang <li...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/92de2b2d5f21af38380b437af51e49c7ac142258
Stats:     20 lines in 8 files changed: 0 ins; 1 del; 19 mod

8336039: Doccheck: HTML warnings, broken links and missing files in java.base 
documentation

Reviewed-by: liach, djelinski

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

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

Reply via email to