On 26/06/2018 18:41, Joe Wang wrote:
:Yes, combined with Sherman's suggestion eliminated the need for the new parameter. Here's the updated webrev:http://cr.openjdk.java.net/~joehw/jdk11/8205058/webrev03/
This looks good.Just a minor nit but newStringNoRepl and getBytesNoReply don't need to check if the cause is null.
-Alan