Looks good to me, Thejasvi. Naoto
On 8/1/19 6:42 AM, Thejasvi Voniadka wrote:
Hi, Request you to please review this change. JBS: https://bugs.openjdk.java.net/browse/JDK-8158880 (test/java/time/tck/java/time/format/TCKDateTimeFormatterBuilder.java fail with zh_CN locale) Description: The test " test/java/time/tck/java/time/format/TCKDateTimeFormatterBuilder.java# test_appendZoneText_parseGenericTimeZonePatterns" fails if run on certain locales (such as "ja"). At present the test does not enforce a specific locale for the DateTimeFormatter, and hence the machine's locale is used as the default. This could cause parsing mismatches when run on non-US locales. The fix is to set the locale explicitly in the test. Webrev: http://cr.openjdk.java.net/~vagarwal/8158880/webrev.0
