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