<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