Looks fine if you only changed the testing comment. Xuelei
On 3/31/2012 6:37 AM, Brad Wetmore wrote: > Reposted to same location under ".01". > > Brad > > > On 3/29/2012 7:42 PM, Brad Wetmore wrote: >> Thanks, I won't regenerate the webrev unless someone really wants to see >> it. >> >> Brad >> >> >> On 3/29/2012 7:30 PM, Xuelei Fan wrote: >>> Forgot one thing about the regression test. JSSE test should run in >>> othervm. Would you please add the following lines to the test: >>> >>> * SunJSSE does not support dynamic system properties, no way to >>> * re-use system properties in samevm/agentvm mode. >>> * @run main/othervm NullGetAcceptedIssuers >>> * >>> >>> Thanks, >>> Xuelei >>> >>> On 3/30/2012 10:26 AM, Xuelei Fan wrote: >>>> Looks fine to me. >>>> >>>> Thanks, >>>> Xuelei >>>> >>>> On 3/30/2012 10:20 AM, Brad Wetmore wrote: >>>>> Hi Xuelei, >>>>> >>>>> Can you please rereview this bug, the 7u4 and 8 changes are here: >>>>> >>>>> http://cr.openjdk.java.net/~wetmore/7142172/ >>>>> >>>>> Thanks, >>>>> >>>>> Brad >>>>> >>>> >>>
