On Wed, 9 Apr 2025 03:51:27 GMT, Renjith Kannath Pariyangad 
<rkannathp...@openjdk.org> wrote:

>> test/jdk/java/awt/print/PrinterJob/PrintNullString.java line 64:
>> 
>>> 62:                 .columns(45)
>>> 63:                 .build();
>>> 64:         new PrintNullString();
>> 
>> Minor: Don't you think It's a bit unusual to have the test be executed with 
>> the builder of the class? Seems a bit odd to me, but if you prefer it, it's 
>> fine as is
>
> This **builder** is for PassFailJFrame (user instruction window).

@Renjithkannath apologies, I meant the constructor, line 64 😄

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24501#discussion_r2034919842

Reply via email to