<i18n dev> [8]Review request for 8004240 and 8008577

[Naoto Sato]

Thank you for the review. Please see my comments embedded below.

On 2/24/13 6:16 PM, Masayoshi Okutsu wrote:
> For 8004240, I think adapterPreference should be changed to an
> unmodifiable List because getAdapterPreference() is called often.

Agree. Will change the impl.

>
> For 8008577, I think it's a spec change to 6336885, and that CCC
> approval is required. And should the CLDR adapter be initialized and be
> removed from the list if the initialization failed?

I think which locale provider adapters are implemented (and enabled) 
depends on each JRE's implementation, not the spec. However, since this 
is going to change Oracle's JRE behavior (even from yet released jdk8 
beta builds), I will file a CCC request for this.

Anyway, I think I'll need to split this review request into two. Will do 
it shortly.

Naoto

>
> Masayoshi
>
>
> On 2/23/2013 6:15 AM, Naoto Sato wrote:
>> Hello,
>>
>> Please review the changes for the following CR:
>>
>> 8004240 : Return value from getAdapterPrefence() can be modified
>> (http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8004240)
>> 8008577 : Enable CLDR LocaleProviderAdapter by default
>> (http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8008577)
>>
>> The webrev is located at:
>> http://cr.openjdk.java.net/~naoto/8004240.8008577/webrev.00/
>>
>> Naoto
>