On Thu, 22 Feb 2024 16:55:17 GMT, Alexander Zuev <[email protected]> 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