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

[Naoto Sato]

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
>