On Tue, 2 Sep 2025 14:35:47 GMT, Mikhail Yankelevich <myankelev...@openjdk.org> 
wrote:

>> Vanitha B P has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   modified tzid variable
>
> test/jdk/java/util/TimeZone/DefaultTimeZoneTest.java line 111:
> 
>> 109:         panel.setLayout(new BoxLayout(panel, BoxLayout.PAGE_AXIS));
>> 110:         // Time zone ID label
>> 111:         var tzid = new JLabel("Time zone ID: " + 
>> SDF.getTimeZone().getID(), SwingConstants.CENTER);
> 
> minor: do you think changing it to a more intuitive `timeZoneID` would make 
> it easier to read? If you want to leave it as is - I don't mind

Done, changed to timeZoneID.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27043#discussion_r2316501052

Reply via email to