The bug references parseStrict() but the test does not. Is the builder already set to parseStrict() ? Anyway, if the bug is to be clearly squished, parseStrict() should appear somewhere. Stephen
On Tue, 10 Sep 2019 at 23:42, Joe Wang <huizhe.w...@oracle.com> wrote: > > +1, looks good to me. > > Best regards, > Joe > > On 9/10/19 2:20 PM, naoto.s...@oracle.com wrote: > > Hello, > > > > Please review the fix to the following issue: > > > > https://bugs.openjdk.java.net/browse/JDK-8230136 > > > > The proposed changeset is located at: > > > > http://cr.openjdk.java.net/~naoto/8230136/webrev.00/ > > > > The fix is to correct the condition of the invalid case, as suggested > > in the bug report. > > > > Naoto >