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