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

naoto.sato at oracle.com naoto.sato at oracle.com
Mon May 6 19:40:51 UTC 2019


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