Re: Request for Review : JDK-8071929 -Locale.getISOCountries() has inconsistent behaviour for "AN", "BU" and "CS" country codes.

[Masayoshi Okutsu]

Hi Rachna,

It's still unclear for me how this addition will be useful for 
applications. But here are my comments on the implementation.

src/java.base/share/classes/java/util/LocaleISOData.java:
- iso3166CodesMap has to use a ConcurrentHashMap.
- The following code should use AssertionError rather than assert.

509 default:
510 // should not happen
511 assert false;
512 return Collections.emptySet();

- Another implementation option would be to move the new get* methods 
and the cache to Locale.IsoCountryCode so that the default case can be 
avoided.

Thanks,
Masayoshi

On 11/29/2016 6:07 PM, Rachna Goel wrote:
Hi,

Please review fix for JDK-8071929.

Bug : https://bugs.openjdk.java.net/browse/JDK-8071929

patch : http://cr.openjdk.java.net/~rgoel/JDK_8071929/webrev.02/

Fix is to remove obsolete country code "AN" and provide support for 
retrieving of ISO3166-1 alpha-2,  ISO3166-1 alpha-3, ISO3166-3 country 
codes.

Thanks,
Rachna