RFR: 8311968: Clarify Three-letter time zone IDs in java.util.TimeZone [v2]

Justin Lu jlu at openjdk.org
Thu Jul 13 18:41:49 UTC 2023


On Thu, 13 Jul 2023 18:29:05 GMT, Naoto Sato <naoto at openjdk.org> wrote:

>> Justin Lu has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review: update wording to include mappings
>
> src/java.base/share/classes/java/util/TimeZone.java line 133:
> 
>> 131:  * for multiple time zones (for example, "CST" could be U.S. "Central Standard
>> 132:  * Time" and "China Standard Time"), and the Java platform can then only
>> 133:  * recognize one of them. The full list of the deprecated IDs can be viewed
> 
> "deprecated IDs and their mappings" may read better, as SHORT_IDS is a map

Good point, clarified the PR and CSR

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14861#discussion_r1262929615


More information about the core-libs-dev mailing list