> On May 4, 2020, at 9:27 AM, naoto.s...@oracle.com wrote:
> 
> Hi Mikael,
> 
> I took a look at i18n related files. It looks good overall.
> 
> One nit in java/nio/charset/Charset/DefaultCharsetTest.java: If the test is 
> only applicable to linux (@requires os.family == "linux" in jtreg tag after 
> the change), the condition isLinux() is no longer needed at line 73.

Good catch! Fixed in the upcoming webrev.

Thank you for the review!

Cheers,
Mikael

Reply via email to