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 On Mar 30, 2014, at 23:27, Wang Weijun <weijun.w...@oracle.com> wrote: > Hi > > Please take a review at > > http://cr.openjdk.java.net/~weijun/8038754/webrev.00/ > > Before JDK-8031003, the debug message inside native JGSS is printed out on > the Java side using > > System.out.println(msg); > > but after it, it is printed on the C side using > > printf(msg); > > and no newline will be automatically added. Therefore output are all packed > into one line like > > [GSSLibStub_acquireCred] pName=4300612336, usage=2[GSSLibStub_acquireCred] > pCred=4298497488PROCISFUN:true > > and ReplayCacheTestProc cannot find the test result "PROCISFUN:true". > > The fix adds the newline char. > > No new test. > > Thanks > Max >