RFR: 8285306: Fix typos in java.desktop [v10]

Alexey Ivanov aivanov at openjdk.org
Thu Sep 22 17:03:27 UTC 2022


On Thu, 22 Sep 2022 01:35:49 GMT, SWinxy <duke at openjdk.org> wrote:

>> Magnus Ihse Bursie has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 49 commits:
>> 
>>  - Merge branch 'master' into typos-in-java.desktop
>>  - Revert changes to glext.h
>>  - Revert changes to multiVis.c and wsutils.h
>>  - Merge branch 'master' into typos-in-java.desktop
>>  - Revert changes in libjpeg
>>  - Revert changes in libfreetype
>>  - Update src/java.desktop/windows/classes/sun/awt/windows/WPrinterJob.java
>>    
>>    Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>>  - Update src/java.desktop/unix/classes/sun/awt/X11/XBaseMenuWindow.java
>>    
>>    Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>>  - Update src/java.desktop/unix/classes/sun/awt/X11/XBaseMenuWindow.java
>>    
>>    Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>>  - Update src/java.desktop/share/classes/sun/swing/SwingUtilities2.java
>>    
>>    Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>>  - ... and 39 more: https://git.openjdk.org/jdk/compare/cddd6def...2850610d
>
> src/java.desktop/share/classes/sun/awt/SunToolkit.java line 1434:
> 
>> 1432: 
>> 1433:     /**
>> 1434:      * Parameterless version of realsync which uses default timeout (see DEFAULT_WAIT_TIME).
> 
> Suggestion:
> 
>      * Parameterless version of realsync which uses the {@link #DEFAULT_WAIT_TIME default timeout}.
> 
> I mean this suggestion [might not be warranted](https://github.com/openjdk/jdk/pull/8328/files#r872596373).

I'm for a clearer message:

Parameterless version of `{@code realsSync}` which uses _the_ default timeout of `{@link #DEFAULT_WAIT_TIME}`.

Let us address this in a separate issue.

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

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



More information about the client-libs-dev mailing list