Hi Sherman, This looks fine to me.
Does the current code fail both of the added checks in the test? Thanks, Brian On Sep 18, 2018, at 5:14 PM, Xueming Shen <xueming.s...@oracle.com> wrote: > Please help codereview the change for JDK-8210899. > > issue: https://bugs.openjdk.java.net/browse/JDK-8210899 > webrev: http://cr.openjdk.java.net/~sherman/8210899/webrev