On Thu, 10 Dec 2020 23:07:52 GMT, Naoto Sato <na...@openjdk.org> wrote:

>> Magnus Ihse Bursie has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Move to share/data, and move jdwp.spec to java.se
>
> Reviewed changes to `characterdata`, `charsetmapping`, `cldr`, `currency`, 
> `lsrdata`, `tzdata`, and `unicodedata` with minor comment. Looks good overall.

I think this is almost ready to be pushed, but I'd like to have someone from 
the client team review the changes for java.desktop as well. @prrace Any change 
you can have a look at this?

> test/jdk/java/util/Locale/LSRDataTest.java line 57:
> 
>> 55:     // path to the lsr file from the make folder, this test relies on the
>> 56:     // relative path to the file in the make folder, considering
>> 57:     // test and make will always exist in the same jdk layout
> 
> The comment refers to the "make" folder. (line 55 as well).

Fixed. Thank you for noticing!

-------------

PR: https://git.openjdk.java.net/jdk/pull/1611

Reply via email to