<i18n dev>  Request for review: 8000986: Split java.util.spi.CalendarDataProvider into week parameters and field names portions
masayoshi.okutsu at oracle.com
Fri Nov 9 17:08:40 PST 2012
Thanks. I added CalendarNameProviderImpl to GenericTest.java. The test
I've updated webrev. No other changes.
On 11/10/2012 4:41 AM, Naoto Sato wrote:
> I have just one comment. Although it won't affect the test result,
> test/java/util/PluggableLocale/GenericTest.java should include the
> available locales from CalendarNameProviderImpl for consistency.
> On 11/9/12 7:59 AM, Masayoshi Okutsu wrote:
>> Please review the changes for:
>> In addition to the split, I've improved CalendarDataUtility which now
>> gets the week parameters through the Pool mechanism.
>> This one should fix the following concrete problem:
>> I've manually tested this one, but I didn't take the test case because
>> it depends on localization data. We need to rely on the test case
>> developed by the SQE team.
More information about the i18n-dev