<i18n dev> [13] RFR: CONFIG level logging statements printed in CLDRCalendarDataProviderImpl.java even when default log Level is INFO

[naoto.sato at oracle.com]

Thanks, Nishit.

I'd still like to ask for a review from a Reviewer.

Naoto

On 2/20/19 12:33 AM, Nishit Jain wrote:
> Hi Naoto,
> 
> Thanks for the explanation. Change looks fine to me.
> 
> Regards,
> Nishit Jain
> On 19-02-2019 22:51, Naoto Sato wrote:
>> Hi Nishit,
>>
>> The reason is that "US" is the only required locale in the JDK (cf. 
>> Locale.getAvailableLocales(). In fact, initially I supplied "001" with 
>> it, as it means the "world" in CLDR, but it broke some existing tests. 
>> "001" returns "MONDAY" as the first day of week, whereas "SUNDAY" in 
>> US. For the compatibility reason, I had to resort to "US". I am not 
>> sure we want to hardcode "1" in this case without any convincing reason.
>>
>> Naoto
>>
>> On 2/19/19 6:37 AM, Nishit Jain wrote:
>>> Hi Naoto,
>>>
>>> Why is the default region set to "US" if there is no region specified 
>>> in the locale? is this the default behavior of "first day of week" 
>>> and "minimal days in first week" when a region is missing or the 
>>> default behavior is that it returns "1"? Can't we just return "1" 
>>> instead of setting the region to "US"?
>>>
>>> Regards,
>>> Nishit Jain
>>> On 16-02-2019 04:25, Naoto Sato wrote:
>>>> Hello,
>>>>
>>>> Please review the fix to the following issue:
>>>>
>>>> https://bugs.openjdk.java.net/browse/JDK-8218960
>>>>
>>>> The proposed changeset is located at:
>>>>
>>>> http://cr.openjdk.java.net/~naoto/8218960/webrev.00/
>>>>
>>>> The CONFIG message was generated because 
>>>> CLDRCalendarDataProviderImpl was returning null for locales without 
>>>> region. Use "US" as the default region in such a case.
>>>>
>>>> Naoto
>>>
>