Thanks for the review, I updated the webrev[1] with simplified test and ensure other cases in boundary won’t be causing trouble by only take meaningful tokens.
This fix is more defensive and allow anchor to be ignored when it’s meaningless. [1] http://cr.openjdk.java.net/~henryjen/jdk/8240629.1/webrev/ Cheers, Henry > On Mar 8, 2020, at 1:54 AM, Alan Bateman <alan.bate...@oracle.com> wrote: > > On 06/03/2020 22:40, Henry Jen wrote: >> Hi, >> >> Please review the webrev[1] fix JDK-8240629 reported earlier by Robert. >> >> http://cr.openjdk.java.net/~henryjen/jdk/8240629.0/webrev/ >> > The changes and test look okay. > > -Alan
