Hi Roger,

Thank you for the review. Wrapped the long lines as suggested, along with some typo fixes in the comments:

https://cr.openjdk.java.net/~naoto/8244245/webrev.01/

Naoto

On 5/7/20 7:43 AM, Roger Riggs wrote:
Hi Naoto,

Looks fine with a small source edit below.

TestUnicodeExtension.java: Please wrap the excessively long lines; string concatination will put them together for the test.

Thanks, Roger


On 5/6/20 4:44 PM, naoto.s...@oracle.com wrote:
Hello,

Please review the fix to the following issue:

https://bugs.openjdk.java.net/browse/JDK-8244245

The CSR and proposed changeset are located at:

https://bugs.openjdk.java.net/browse/JDK-8244246
https://cr.openjdk.java.net/~naoto/8244245/webrev.00/

This stems from the closed issue (https://bugs.openjdk.java.net/browse/JDK-8243162), and the rationale for this fix is discussed there.

Naoto


Reply via email to