On Wed, 1 Mar 2023 23:06:10 GMT, Steven R. Loomis <d...@openjdk.org> 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, CLDRConverter tool needs to comb >> through the locale files based on the `coverageLevels.txt` file, (and the >> ones we already included as of v42). Confirmed the same set of locales is >> generated before and after this modification. > > 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: (v1, v2) -> v2, > > this will grab all listed entries. Right now, that will get you basic and > above. Ideally you would include if `v1` is one of > `(basic|moderate|modern|comprehensive)` — I'm proposing to add `core` or > maybe even `undefined` locales in this list (though no consensus yet) OK, will filter explicitly for those levels. ------------- PR: https://git.openjdk.org/jdk/pull/12812
