> 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: Implement Roger's review ------------- Changes: - all: https://git.openjdk.org/jdk/pull/27596/files - new: https://git.openjdk.org/jdk/pull/27596/files/3720e24a..fb6b1dcc Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=27596&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=27596&range=01-02 Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 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
