<i18n dev> Review Request:JDK-8163350-LocaleProviderAdapter Preference list retrieved is wrong, when -Djava.locale.providers=COMPAT
Naoto Sato
naoto.sato at oracle.com
Tue Aug 9 16:09:33 UTC 2016
Hi Rachna,
I should've caught it at the internal review, but here are some cosmetic
comments to the test case.
- I'd prefer a blank line between import statements and the main method.
- The argument to main() should be (String[] args)
- Indentation is incorrect at line 39
- Needs parens for the if statement at line 41.
Naoto
On 8/9/16 3:51 AM, Rachna Goel wrote:
> Hi,
>
> Please review fix for JDK-8163350.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8163350
>
> Webrev: http://cr.openjdk.java.net/~rgoel/jdk-8163350/webrev.02/
>
> Fix: If user specifies either CLDR or COMPAT via System Property,
> FALLBACK should not be implicitly included.
>
> It is meant to be included when no ResourceBundleBased Adapters are
> available.
>
> Thanks,
>
> Rachna
>
More information about the i18n-dev
mailing list