RFR: 8334016: Make PrintNullString.java automatic [v5]
Alexey Ivanov
aivanov at openjdk.org
Thu May 22 10:37:59 UTC 2025
On Wed, 14 May 2025 04:07:08 GMT, Renjith Kannath Pariyangad <rkannathpari at openjdk.org> wrote:
>> Hi Reviewers,
>>
>> I have updated the test and made it semi-automatic (need to click "Print") . Test will initiate a print and result will be generated automatic.
>>
>> Please review and let me know your suggestions.
>
> Renjith Kannath Pariyangad has updated the pull request incrementally with one additional commit since the last revision:
>
> Moved main fuction to top
Looks good to me except for a minor nit.
test/jdk/java/awt/print/PrinterJob/PrintNullString.java line 56:
> 54: throw new RuntimeException("Printer not configured or available.");
> 55: }
> 56: new PrintNullString();
Suggestion:
new PrintNullString();
Leave a blank line between the skipped condition and the real test code.
-------------
Marked as reviewed by aivanov (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/24501#pullrequestreview-2860700039
PR Review Comment: https://git.openjdk.org/jdk/pull/24501#discussion_r2102225612
More information about the client-libs-dev
mailing list