<i18n dev> RFR: 8303440: The "ZonedDateTime.parse" may not accept the "UTC+XX" zone id [v2]

Jaikiran Pai jpai at openjdk.org
Sat Mar 4 01:40:06 UTC 2023


On Fri, 3 Mar 2023 21:10:57 GMT, Naoto Sato <naoto at openjdk.org> wrote:

>> This change is to fix a regression caused by the fix to [JDK-8234347](https://bugs.openjdk.org/browse/JDK-8234347). The previous fix was simply disabling offset-based parsing if the parser was text-based. Instead, check if it is an offset or not by explicitly comparing the character with '+'/'-' and continue offset parsing if it is.
>
> Naoto Sato has updated the pull request incrementally with one additional commit since the last revision:
> 
>   now() -> of() to avoid DST

test/jdk/java/time/test/java/time/format/TestUTCParse.java line 27:

> 25:  * @modules jdk.localedata
> 26:  * @bug 8303440
> 27:  * @summary Test parsing "UTC-XX:XX" text works correctly

Hello Naoto, I haven't tried it, but without a `@run testng` instruction, does jtreg have the ability to auto infer this as a testng testcase?

test/jdk/java/time/test/java/time/format/TestUTCParse.java line 48:

> 46:         // Assuming CLDR's SHORT name for "America/Los_Angeles"
> 47:         // produces "UTC\u212208:00"
> 48:         System.setProperty("java.locale.providers", "CLDR");

Since this updates a system property (that's recognized by code even outside of this test), perhaps we should use `@run testng/othervm` to prevent interfering with other tests?

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

PR: https://git.openjdk.org/jdk/pull/12868


More information about the i18n-dev mailing list