On Fri, 27 May 2022 16:19:56 GMT, Naoto Sato <na...@openjdk.org> wrote:
>> XenoAmess has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Merge remote-tracking branch 'stuart-marks/pull/8302' into fix_8284780 >> >> # Conflicts: >> # src/java.base/share/classes/java/util/HashSet.java >> # src/java.base/share/classes/java/util/LinkedHashSet.java >> - Minor terminology fixes: change "item" and "key" to element; remove >> "insertion-ordered" from LinkedHashSet static factory method because >> LHS is implicit insertion-ordered. > > src/java.base/share/classes/sun/util/cldr/CLDRLocaleProviderAdapter.java line > 227: > >> 225: } >> 226: StringTokenizer tokens = new >> StringTokenizer(supportedLocaleString); >> 227: Set<String> tagset = HashSet.newHashSet(tokens.countTokens()); > > Hi @XenoAmess, during the refactoring of the above `StringTokenizer`, it > turned out that this `HashSet` can be replaced with `Set.of()`. So you can > leave this piece as it is, as I will take care of it with JDK-8287340. @naotoj done. > src/java.base/share/classes/sun/util/locale/provider/JRELocaleProviderAdapter.java > line 461: > >> 459: } >> 460: StringTokenizer tokens = new >> StringTokenizer(supportedLocaleString); >> 461: Set<String> tagset = HashSet.newHashSet(tokens.countTokens()); > > Same as above. @naotoj done. ------------- PR: https://git.openjdk.java.net/jdk/pull/8302
