On Wed, 31 Jan 2024 09:22:41 GMT, Tejesh R <t...@openjdk.org> wrote:
>> Renjith Kannath Pariyangad has updated the pull request incrementally with
>> one additional commit since the last revision:
>>
>> Capitalized static final variables
>
> 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.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17609#discussion_r1472645679
