This fix looks fine to me. Thanks.
> On 31 Mar 2017, at 16:34, Sean Mullan <sean.mul...@oracle.com> wrote: > > New webrev uploaded to address your comments: > http://cr.openjdk.java.net/~mullan/webrevs/8175029/webrev.01/ > > --Sean > > On 3/31/17 9:11 AM, Sean Mullan wrote: >> On 3/31/17 9:08 AM, Weijun Wang wrote: >>> Can we just move the code you just added into the public API directly? >>> Looks like it does not reference any internal classes. >> >> Good point, that should work! I'll try that and send out an updated >> webrev later. >> >> --Sean >> >>> >>> --Max >>> >>> On 03/31/2017 08:06 PM, Sean Mullan wrote: >>>> Please review this fix for a StackOverflowError caused by the default >>>> implementation of verify(PublicKey, Provider). >>>> >>>> bug: https://bugs.openjdk.java.net/browse/JDK-8175029 >>>> >>>> webrev: http://cr.openjdk.java.net/~mullan/webrevs/8175029/webrev.00/ >>>> >>>> Thanks, >>>> Sean
