On Wed, 3 Sep 2025 23:46:50 GMT, Chen Liang <[email protected]> wrote:
>> Damon Nguyen has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Add finally block
>
> src/java.desktop/share/classes/java/awt/Robot.java line 792:
>
>> 790: /**
>> 791: * A convenience method that simulates clicking a mouse button by
>> calling {@code mousePress}, {@code mouseRelease},
>> 792: * and {@code waitForIdle}. Invokes {@code waitForIdle} with a
>> default delay of 20 milliseconds after
>
> Suggestion:
>
> * and {@code waitForIdle}. Invokes {@code waitForIdle} with a default
> delay of {@value #DEFAULT_STEP_DELAY} milliseconds after
>
> This renders `20` as a link that links to the constant that users can use.
Updated, thanks.
> src/java.desktop/share/classes/java/awt/Robot.java line 855:
>
>> 853: *
>> 854: * @implSpec Invokes {@link #mouseMove(int, int) mouseMove} with a
>> default
>> 855: * {@link #DEFAULT_STEP_LENGTH step-length} and {@link
>> #DEFAULT_STEP_DELAY step-delay}.
>
> You can use:
>
> * @implSpec Invokes {@link #mouseMove(int, int) mouseMove} with a
> step-length
> * of {@value #DEFAULT_STEP_LENGTH} and a step-delay of {@value
> #DEFAULT_STEP_DELAY}.
>
>
> Renders like:
>
>> Invokes [`mouseMove`]() with a step-length of [`2`]() and a a step-delay of
>> [`20`]().
>
> Same for other sites.
Updated this and in similar spots. Let me know if you see any other issues with
this. Thanks!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26969#discussion_r2331578071
PR Review Comment: https://git.openjdk.org/jdk/pull/26969#discussion_r2331579028
