> Please review this PR which case folds grandfathered BCP47 tags on demand. > (Instead of keeping the case compliant versions stored.) > > This facilitates the cleanup of the `LEGACY` map (which contains a String to > String Array mapping). The 2D String array, creation and populating of the > `LEGACY` map, as well as lower case folding of the legacy tags can now be > removed from the static block of _sun.util.locale.LanguageTag_. > > Legacy tags are a small subset compared to the entire set of valid BCP47 > tags. Reducing the cost for supporting them would be ideal. Instead, legacy > tags can now be checked against and folded on demand via a switch statement. > > The commented grandfathered syntax is removed because the values themselves > already exist as code, and the exact case can be found in the provided RFC > link.
Justin Lu has updated the pull request incrementally with one additional commit since the last revision: implemenet Naoto's review ------------- Changes: - all: https://git.openjdk.org/jdk/pull/27596/files - new: https://git.openjdk.org/jdk/pull/27596/files/710d7737..3720e24a Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=27596&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=27596&range=00-01 Stats: 40 lines in 2 files changed: 15 ins; 15 del; 10 mod Patch: https://git.openjdk.org/jdk/pull/27596.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/27596/head:pull/27596 PR: https://git.openjdk.org/jdk/pull/27596
