On 03/16/2017 08:04 PM, Sergey Kuksenko wrote:
> http://cr.openjdk.java.net/~skuksenko/corelibs/utils/8176894/webrev.00/

Looks good. I assume there are enough tests to cover these paths?

*) Missing whitespace (multiple times in the similar line):

 if(replaceOld) {

Thanks,
-Aleksey

Reply via email to