On Mon, 16 Jun 2025 14:14:05 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   javadoc
>
> test/jdk/javax/swing/ImageIcon/ImageIconNullImageTest.java line 40:
> 
>> 38:             ImageIcon icon = new ImageIcon();
>> 39:             icon.setImage(null);
>> 40:         });
> 
> `invokeAndWait` isn't needed here, you can run the test code on the main 
> thread.
> 
> A comment that no NPE is expected would be good to clarify what the test 
> expects.

ok

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25767#discussion_r2150226985

Reply via email to