Hello Naoto.
I appreciate your comments.
I applied following changes:
* MS950.nr and TestMS950.java data were sorted by Unicode order
* Added some comments into TestMS950.java
* Change comment on MS950.map
Could you review the fix ?
Bug: https://bugs.openjdk.java.net/browse/JDK-8232161
Change: https://cr.openjdk.java.net/~itakiguchi/8232161/webrev.02/
Thanks,
Ichiroh Takiguchi
On 2020-03-03 10:31, naoto.s...@oracle.com wrote:
Hi Takiguchi-san,
A few comments:
- I'd recommend sorting the entries in MS950.nr and test data in
TestMS950.java for readability.
- Add some comment about the objective in the test. It'd be hard for
engineers who have no previous knowledge to these bytes.
Naoto
On 3/2/20 9:33 AM, Ichiroh Takiguchi wrote:
Hello.
Could you review the fix ?
Bug: https://bugs.openjdk.java.net/browse/JDK-8232161
Change: https://cr.openjdk.java.net/~itakiguchi/8232161/webrev.01/
CSR 8233385 [1] was approved.
[1] https://bugs.openjdk.java.net/browse/JDK-8233385
Thanks,
Ichiroh Takiguchi
IBM Japan, Ltd.