On Thu, 22 Feb 2024 16:55:17 GMT, Alexander Zuev <kiz...@openjdk.org> wrote:

>> Alisen Chung has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   update test strings and newlines, remove else
>
> test/jdk/java/awt/TrayIcon/DisposeInActionEventTest/DisposeInActionEventTest.java
>  line 89:
> 
>> 87:         frame.setVisible(true);
>> 88: 
>> 89:         System.setProperty("sun.awt.exception.handler\n",
> 
> Why is there \n in the property name?

i wasn't sure how to do the key value pairs for setProperty. Should there just 
be a space between sun.awt.exception.handler and 
DisposeInActionEventTest$EDTExceptionHandler?

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

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

Reply via email to