<i18n dev> Review request : JDK-8158504 - test/sun/util/locale/provider/Bug8038436.java fails

Naoto Sato naoto.sato at oracle.com
Thu Jun 23 16:05:55 UTC 2016

Hi Rachna,

The test needs to be modified to ensure locales only for ROOT and en_US* 
locales for java.base module. Currently it would pass if 
getAvailableLocales() included English locales other than US ones, say 
en_GB. I should have corrected it, but apparently I overlooked.

 > if (nonEnglishLocales.size() > 0) {

You could use !isEmpty() instead.


On 6/23/16 2:09 AM, Rachna Goel wrote:
> Hi,
> Please review fix for JDK-8158504.
> Bug : https://bugs.openjdk.java.net/browse/JDK-8158504
> Webrev : http://cr.openjdk.java.net/~nishjain/rgoel/8158504/webrev.03/
> Fix : Removed java.ext.dirs System property as it is no longer supported
> in JDK9 and used "-limitmods java.base"  to load only java.base.
>         (To hide jdk.localedata module).
>         JRE is COMPAT in JDK9.
>         outputted some useful information if test fails somewhere.

More information about the i18n-dev mailing list