RFR: 8320608: Many jtreg printing tests are missing the @printer keyword [v2]
Phil Race
prr at openjdk.org
Thu Nov 30 18:18:10 UTC 2023
On Thu, 30 Nov 2023 15:59:29 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> Phil Race has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8320608
>
> test/jdk/java/awt/print/PageFormat/SmallPaperPrinting.java line 73:
>
>> 71: job.print();
>> 72: }
>> 73: catch (PrinterException e) {
>
> Suggestion:
>
> } catch (PrinterException e) {
>
> The `catch` keyword should be on the line with the closing brace of the `try {}` block.
>
> It can be addressed in [JDK-8320671](https://bugs.openjdk.org/browse/JDK-8320671) when the test is updated.
fixed anyway
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16785#discussion_r1411095104
More information about the client-libs-dev
mailing list