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