On Thu, 22 Sep 2022 01:35:49 GMT, SWinxy <d...@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-...@users.noreply.github.com>
>>  - Update src/java.desktop/unix/classes/sun/awt/X11/XBaseMenuWindow.java
>>    
>>    Co-authored-by: Alexey Ivanov 
>> <70774172+aivanov-...@users.noreply.github.com>
>>  - Update src/java.desktop/unix/classes/sun/awt/X11/XBaseMenuWindow.java
>>    
>>    Co-authored-by: Alexey Ivanov 
>> <70774172+aivanov-...@users.noreply.github.com>
>>  - Update src/java.desktop/share/classes/sun/swing/SwingUtilities2.java
>>    
>>    Co-authored-by: Alexey Ivanov 
>> <70774172+aivanov-...@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

Reply via email to