[15] RFR: 8246662: Test java/time/test/java/time/format/TestUnicodeExtension.java failed on japanese locale.

[Roger Riggs]

Hi Naoto,

Since the default locale is being changed even briefly, the test should 
be run in /othervm.
Add:  @run testng/othervm ...

DateTimeFormatter:1498:
    Can the optimization be retained in the case where z.equals(zone)?
    Move it down to 1508+ and check the zone vs z.

Thanks, Roger



On 6/8/20 5:06 PM, naoto.sato at oracle.com wrote:
> Hello,
>
> Please review the fix to the following issue:
>
> https://bugs.openjdk.java.net/browse/JDK-8246662
>
> The proposed changeset is located at:
>
> https://cr.openjdk.java.net/~naoto/8246662/webrev.00/
>
> This is a regression caused by the fix to JDK-8244245, where the 
> locale related fields in the formatter are now overridden by 
> localizedBy() method. It was only comparing the locale object for the 
> fast path, but it was not sufficient.
>
> Naoto