[13] RFR: 8220037: Inconsistencies of generated timezone files between Windows and Linux

Roger Riggs Roger.Riggs at oracle.com
Tue May 7 13:38:20 UTC 2019


Looks fine.

Thanks, Roger

On 05/06/2019 03:40 PM, naoto.sato at oracle.com wrote:
> Thanks, Roger. Webrev is modified accordingly:
>
> http://cr.openjdk.java.net/~naoto/8220037/webrev.01/
>
> Naoto
>
> On 5/6/19 11:59 AM, Roger Riggs wrote:
>> Hi Naoto,
>>
>> CLDRConverter:
>>   358-271: Is a lambda viable here?  Not significantly different, 
>> just a more contemporary style.
>>      (o1, o2) -> {...}.
>>
>> CLDRDisplayNamesTest.java:
>>   136-137, 155, 158:  Another option is to have the stream return the 
>> count
>> of the number of errors and use peek to print the markers.
>>
>> errors += List.of(Locale.ROOT,
>>          Locale.CHINA,
>>          Locale.GERMANY,
>>          Locale.JAPAN,
>>          Locale.UK,
>>          Locale.US,
>>          Locale.forLanguageTag("hi-IN"),
>>          Locale.forLanguageTag("es-419")).stream()
>>          .peek(System.out::println)
>>          .map(l -> DateFormatSymbols.getInstance(l).getZoneStrings())
>>          .flatMap(zoneStrings -> Arrays.stream(zoneStrings))
>>          .filter(namesArray -> Arrays.stream(namesArray)
>>                  .anyMatch(aName -> 
>> aName.equals(NO_INHERITANCE_MARKER)))
>>          .peek(marker -> {
>>              System.err.println("No inheritance marker detected with 
>> tzid: " + marker[0]);
>>          })
>>          .count();
>>
>>
>> 151:  findAny() short circuits the count and stream so no more than 1 
>> error would be counted.
>>
>> $.02, Roger
>>
>>
>>
>> On 05/06/2019 01:24 PM, naoto.sato at oracle.com wrote:
>>> Hello,
>>>
>>> Please review the fix to the following issue:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8220037
>>>
>>> The proposed fix is located at:
>>>
>>> http://cr.openjdk.java.net/~naoto/8220037/webrev.00/
>>>
>>> The inconsistency comes from the different enumeration order of CLDR 
>>> source files on each platform. Fix is to sort its order uniquely.
>>>
>>> Naoto
>>



More information about the core-libs-dev mailing list