On Fri, 16 Feb 2024 21:30:21 GMT, Alisen Chung <[email protected]> wrote:

>> Root cause of the test failure was fixed with 
>> https://bugs.openjdk.org/browse/JDK-8316931, updating this test since the 
>> other fix also included a test update.
>
> Alisen Chung has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   remove unused var

test/jdk/java/awt/TrayIcon/DisposeInActionEventTest/DisposeInActionEventTest.java
 line 52:

> 50:         if (!SystemTray.isSupported()) {
> 51:             throw new jtreg.SkippedException("The test cannot be run 
> because SystemTray is not supported.");
> 52:         } else {

Probably else is not required here, you can just remove it and put your code. 
Indentations can be reduced if you take out from else.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17838#discussion_r1494278916

Reply via email to