On Wed, 31 Jan 2024 10:49:05 GMT, Renjith Kannath Pariyangad <rkannathp...@openjdk.org> wrote:
>> test/jdk/java/awt/print/PrinterJob/PrintNullString.java line 142: >> >>> 140: } catch (NullPointerException e) { >>> 141: g.drawString("caught expected NPE for null String, >>> int", 20, 40); >>> 142: }/* catch (Exception e) { >> >> I guess u can remove these instead of commenting? Any reason for commenting >> these instead of removing? > > These lines are commented in existing code. Felt it may be useful for further > debugging in case of failure, so kept as it is. Ok, else the fix is fine for me. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17609#discussion_r1472660678