Re: RFR: 8306323: Update license files in CLDR v43

2023-04-18 Thread Steven R . Loomis
On Tue, 18 Apr 2023 18:40:03 GMT, Naoto Sato wrote: > The upgrade to CLDR v43 was missing the license-related file updates. Here > are the supplemental updates. Marked as reviewed by srl...@github.com (no known OpenJDK username). - PR Review:

Re: RFR: 8296248: Update CLDR to Version 43.0

2023-04-13 Thread Steven R . Loomis
On Thu, 13 Apr 2023 20:47:39 GMT, Steven R. Loomis wrote: >> Upgrading the CLDR to [version >> 43](https://cldr.unicode.org/index/downloads/cldr-43). This semi-annual >> release is their `limited-submission` release so I would not expect >> regressions caused by form

Re: RFR: 8296248: Update CLDR to Version 43.0

2023-04-13 Thread Steven R . Loomis
On Thu, 13 Apr 2023 20:20:02 GMT, Naoto Sato wrote: > Upgrading the CLDR to [version > 43](https://cldr.unicode.org/index/downloads/cldr-43). This semi-annual > release is their `limited-submission` release so I would not expect > regressions caused by formatting changes as we had in

Re: RFR: 8305400: ISO 4217 Amendment 175 Update

2023-03-31 Thread Steven R . Loomis
On Fri, 31 Mar 2023 21:38:31 GMT, Justin Lu wrote: > Please review the ISO 4217 amendment 175 update. > > There are no meaningful code changes, but the version number should be > updated accordingly to be in sync. Marked as reviewed by srl...@github.com (no known OpenJDK username).

Re: RFR: 8305400: ISO 4217 Amendment 175 Update

2023-03-31 Thread Steven R . Loomis
On Fri, 31 Mar 2023 21:38:31 GMT, Justin Lu wrote: > Please review the ISO 4217 amendment 175 update. > > There are no meaningful code changes, but the version number should be > updated accordingly to be in sync. Do you track the legal tender date? Because that did change, for CLDR

Re: RFR: 8303833: java.util.LocaleISOData has wrong comments for 'Norwegian Bokmål' and 'Volapük'

2023-03-08 Thread Steven R . Loomis
On Wed, 8 Mar 2023 19:57:43 GMT, Eirik Bjorsnos wrote: > Please review this comment-only PR which fixes incorrect language names > 'Norwegian Bokmål' and 'Volapük' in the comments in LocaleISOData: > > `+ "nb" + "nob" // Norwegian Bokm?l` > `+ "vo" + "vol" // Volap?k` > > These encoding

Re: RFR: 8303472: Display name for region TR [v2]

2023-03-02 Thread Steven R . Loomis
On Wed, 1 Mar 2023 23:45:47 GMT, Justin Lu wrote: >> This PR changes the English name for the region `TR`, from `Turkey` to >> `Türkiye`. Although this change is included in the upcoming CLDR v43, it >> should be applied as a spot change so that it can be back-ported properly >> (As it is a

Re: RFR: 8303039: Utilize `coverageLevels.txt` [v2]

2023-03-01 Thread Steven R . Loomis
On Thu, 2 Mar 2023 01:03:20 GMT, Naoto Sato wrote: >> This is a pre-requisite for supporting CLDR v43, where they combine `seeds` >> locales with `common` locales >> (https://cldr.unicode.org/index/downloads/cldr-43#h.7s25aqdv767e). In order >> to have the same coverage level of locales,

Re: RFR: 8303039: Utilize `coverageLevels.txt`

2023-03-01 Thread Steven R . Loomis
On Wed, 1 Mar 2023 23:19:01 GMT, Naoto Sato wrote: >> make/jdk/src/classes/build/tools/cldrconverter/CLDRConverter.java line 1212: >> >>> 1210: a -> >>> Locale.forLanguageTag(a[0].trim().replaceAll("_", "-")), >>> 1211: a -> a[1].trim(), >>> 1212:

Re: RFR: 8303039: Utilize `coverageLevels.txt`

2023-03-01 Thread Steven R . Loomis
On Wed, 1 Mar 2023 23:18:59 GMT, Naoto Sato wrote: >> is this from the alpha2 drop? or v42's? > > Yes, this is the file from the released CLDR v42. We have not integrated v43 > yet. this is great groundwork. - PR: https://git.openjdk.org/jdk/pull/12812

Re: RFR: 8303039: Utilize `coverageLevels.txt`

2023-03-01 Thread Steven R . Loomis
On Wed, 1 Mar 2023 23:02:06 GMT, Steven R. Loomis wrote: >> This is a pre-requisite for supporting CLDR v43, where they combine `seeds` >> locales with `common` locales >> (https://cldr.unicode.org/index/downloads/cldr-43#h.7s25aqdv767e). In order >> to have the same c

Re: RFR: 8303039: Utilize `coverageLevels.txt`

2023-03-01 Thread Steven R . Loomis
On Wed, 1 Mar 2023 19:50:56 GMT, Naoto Sato wrote: > This is a pre-requisite for supporting CLDR v43, where they combine `seeds` > locales with `common` locales > (https://cldr.unicode.org/index/downloads/cldr-43#h.7s25aqdv767e). In order > to have the same coverage level of locales,

Re: RFR: 8303039: Utilize `coverageLevels.txt`

2023-03-01 Thread Steven R . Loomis
On Wed, 1 Mar 2023 19:50:56 GMT, Naoto Sato wrote: > This is a pre-requisite for supporting CLDR v43, where they combine `seeds` > locales with `common` locales > (https://cldr.unicode.org/index/downloads/cldr-43#h.7s25aqdv767e). In order > to have the same coverage level of locales,

Re: RFR: 8284840: Update CLDR to Version 42.0

2022-10-22 Thread Steven R . Loomis
On Sat, 22 Oct 2022 08:14:00 GMT, Alan Bateman wrote: > > Yes. These translation changes affect formatting. We don't usually file a > > CSR for such changes, but cover them in our release notes. > > Indeed and periodically CLDR upgrades do cause breakage somewhere, often it > will be a

Re: RFR: 8284840: Update CLDR to Version 42.0

2022-10-21 Thread Steven R . Loomis
On Fri, 21 Oct 2022 16:55:28 GMT, Naoto Sato wrote: > This is to update the CLDR data from version 41 to version 42. The vast > majority of the changes are basically replacing the CLDR data, along with > tools/testcase alignments to those upstream changes: > >

Re: RFR: 8289227: Support for BCP 47 Extension T - Transformed Content [v6]

2022-08-01 Thread Steven R . Loomis
On Mon, 1 Aug 2022 21:27:47 GMT, Steven R. Loomis wrote: >> Naoto Sato has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Use `assertThrows` > > test/jdk/java/util/Locale/bcp47/TExtensionTests.java line 80

Re: RFR: 8289227: Support for BCP 47 Extension T - Transformed Content [v6]

2022-08-01 Thread Steven R . Loomis
On Sat, 30 Jul 2022 20:52:49 GMT, Naoto Sato wrote: >> This PR is to propose supporting the `T` extension to the BCP 47 to which >> `java.util.Locale` class conforms. There are two extensions to the BCP 47, >> one is `Unicode Locale Extension` which has been supported since JDK7, the >> other