Hi Naoto,
Looks fine.
I don't think it needs to be run in 'othervm' since it does not pollute
the VM it is running in.
It might run a few milliseconds faster without that. No updated webrev
needed.
Regards, Roger
On 01/11/2019 02:15 PM, naoto.s...@oracle.com wrote:
Hello,
Please review the fix to the following issue:
https://bugs.openjdk.java.net/browse/JDK-8216140
The proposed changeset is located at:
http://cr.openjdk.java.net/~naoto/8216140/webrev.00/
This issue was previously discussed in
https://bugs.openjdk.java.net/browse/JDK-8150449 and was closed as
"not an issue" for JDK8. Unicode Consortium then made a corrigendum to
Unicode 7 that recommends passing through of the code point. To
conform to the standard, the offending test/error return has been
removed, based on the recommendation in the previous consideration on
8150449.
Naoto