On Tue, 26 Jul 2022 19:03:24 GMT, Joe Wang <jo...@openjdk.org> wrote:
>> Naoto Sato has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains 12 additional >> commits since the last revision: >> >> - Modified javadoc of the transformed conent >> - Merge branch 'master' into JDK-8289227-T-ext >> - Removed unnecessary `contains()` check >> - IllformedLocaleEx -> LocaleSyntaxEx >> - SystemProperty tests >> - Revived returning Optional >> - Some clean-ups, including making Extension a sealed class. >> - Bring the specialized methods back >> Some documentation fixes >> - Using Optional >> - FieldSeparators()/FieldSubtag() -> Fields() >> - ... and 2 more: https://git.openjdk.org/jdk/compare/b512a057...780f712e > > src/java.base/share/classes/java/util/Locale.java line 265: > >> 263: * field separator (one alpha + one digit), followed by one or more >> subtags of the length 3 to 8, >> 264: * each delimited by a hyphen. >> 265: * <p>The transformed content information; namely {@code source} >> language tag and {@code fields} > > typo ";" (s/;/,) Actually, I did mean a semicolon, but a comma may be better in this case. ------------- PR: https://git.openjdk.org/jdk/pull/9620