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 <[email protected]> wrote: > > +1, looks good to me. > > Best regards, > Joe > > On 9/10/19 2:20 PM, [email protected] 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 >
