Improved webrev, that uses Locale.Builder to detect syntactically bad
locales.
Webrev: http://cr.openjdk.java.net/~jjg/8149565/webrev.00/index.html
-- Jon
On 06/07/2018 03:19 PM, Jonathan Gibbons wrote:
Please review a relatively simple fix to a problem that was found a
while back relating to the handling of the -locale option.
The fix is to change some old code, that did not work on all locales
installed in a standard JDK build, with the use of
Locale.forLanguageTag, added in JDK 7. This method supports IETF BCP
47 language tag strings.
The test is removed from the problem list and updated to work
correctly. The primary error in the test as originally written was
that the doclet invocation would create the doclet instance in a
distinct classloader, and thus not share static members, as was
clearly the intent.
JBS: https://bugs.openjdk.java.net/browse/JDK-8149565
Webrev: http://cr.openjdk.java.net/~jjg/8149565/webrev.00/index.html
-- Jon
