Re: RFR 8038754: ReplayCacheTestProc test fails with timeout

2014-03-31 Thread Alan Bateman
On 31/03/2014 01:56, Wang Weijun wrote: Webrev updated at http://cr.openjdk.java.net/~weijun/8038754/webrev.01/ 'printf(s); printf("\n")' is now 'printf(s"\n");'. One less call, and seems an extra benefit to force /s/ being a real literal. At least it compiles now. Thanks Max Good sleuthi

Re: Review Request for JDK-8030114: [parfait] warnings from b119 for jdk.src.share.native.sun.security.smartcardio: JNI exception pending

2014-03-31 Thread Vincent Ryan
Your latest webrev looks good. Thanks. On 6 Mar 2014, at 00:53, Valerie (Yu-Ching) Peng wrote: > > I still need a reviewer for integrating the changes for 8030114... Any taker? > Thanks, > Valerie > > On 02/21/14 16:41, Valerie (Yu-Ching) Peng wrote: >> >> Good points, I have updated the web