On Tue, 18 Mar 2025 20:37:43 GMT, Alexey Ivanov <[email protected]> wrote:
>> Manukumar V S has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Review comments fixed : Added SkippedException instead of forcePass(),
>> Changed the position of imports, formatting changes
>
> test/jdk/java/awt/Desktop/MailTest.java line 106:
>
>> 104: if (!desktop.isSupported(Desktop.Action.MAIL)) {
>> 105: String errorMessage = "Action.MAIL is not supported.";
>> 106: throw new SkippedException(errorMessage);
>
> Since the error message is used only once, you can inline the message to the
> constructor.
>
> You may inline `Desktop.getDesktop().isSupported(Desktop.Action.MAIL)`
> without introducing a local variable if you like.
Done
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24072#discussion_r2002311621
