<i18n dev> RFR: 8342550: Log warning for using JDK1.1 compatible time zone IDs for future removal
Justin Lu
jlu at openjdk.org
Mon Jan 13 19:59:35 UTC 2025
On Fri, 10 Jan 2025 23:05:13 GMT, Naoto Sato <naoto at openjdk.org> wrote:
> The use of [three-letter time zone IDs](https://download.java.net/java/early_access/jdk24/docs/api/java.base/java/util/TimeZone.html#three-letter-time-zone-ids-heading) has long been deprecated. For their actual removal in the future, making the runtime emit warnings for using those IDs would be desirable.
>
> ---------
> ### Progress
> - [ ] Change must be properly reviewed (1 review required, with at least 1 [Reviewer](https://openjdk.org/bylaws#reviewer))
> - [x] Change must not contain extraneous whitespace
> - [x] Commit message must refer to an issue
>
>
>
> ### Reviewers without OpenJDK IDs
> * @abdelhak-zaaim (no known openjdk.org user name / role)
>
> ### Reviewing
> <details><summary>Using <code>git</code></summary>
>
> Checkout this PR locally: \
> `$ git fetch https://git.openjdk.org/jdk.git pull/23049/head:pull/23049` \
> `$ git checkout pull/23049`
>
> Update a local copy of the PR: \
> `$ git checkout pull/23049` \
> `$ git pull https://git.openjdk.org/jdk.git pull/23049/head`
>
> </details>
> <details><summary>Using Skara CLI tools</summary>
>
> Checkout this PR locally: \
> `$ git pr checkout 23049`
>
> View PR using the GUI difftool: \
> `$ git pr show -t 23049`
>
> </details>
> <details><summary>Using diff file</summary>
>
> Download this PR as a diff file: \
> <a href="https://git.openjdk.org/jdk/pull/23049.diff">https://git.openjdk.org/jdk/pull/23049.diff</a>
>
> </details>
Fix and associated test both look good to me.
-------------
Marked as reviewed by jlu (Committer).
PR Review: https://git.openjdk.org/jdk/pull/23049#pullrequestreview-2547726687
More information about the i18n-dev
mailing list