Re: RFR: 8259799: vmTestbase/nsk/jvmti/Breakpoint/breakpoint001 is incorrect [v2]

2021-01-14 Thread Chris Plummer
On Thu, 14 Jan 2021 20:32:17 GMT, Leonid Mesnik wrote: >> est vmTestbase/nsk/jvmti/Breakpoint/breakpoint001 has incorrect check of >> strcmp results here: >> >> for (i=0; i> if (strcmp(methNam,METHODS[i][0]) && >> strcmp(methSig,METHODS[i][1])) { >> printf(

Re: RFR: 8259799: vmTestbase/nsk/jvmti/Breakpoint/breakpoint001 is incorrect [v2]

2021-01-14 Thread Serguei Spitsyn
On Thu, 14 Jan 2021 20:32:17 GMT, Leonid Mesnik wrote: >> est vmTestbase/nsk/jvmti/Breakpoint/breakpoint001 has incorrect check of >> strcmp results here: >> >> for (i=0; i> if (strcmp(methNam,METHODS[i][0]) && >> strcmp(methSig,METHODS[i][1])) { >> printf(

Re: RFR: 8259799: vmTestbase/nsk/jvmti/Breakpoint/breakpoint001 is incorrect [v2]

2021-01-14 Thread Igor Ignatyev
On Thu, 14 Jan 2021 20:32:17 GMT, Leonid Mesnik wrote: >> est vmTestbase/nsk/jvmti/Breakpoint/breakpoint001 has incorrect check of >> strcmp results here: >> >> for (i=0; i> if (strcmp(methNam,METHODS[i][0]) && >> strcmp(methSig,METHODS[i][1])) { >> printf(

Re: RFR: 8259799: vmTestbase/nsk/jvmti/Breakpoint/breakpoint001 is incorrect [v2]

2021-01-14 Thread Leonid Mesnik
On Thu, 14 Jan 2021 19:22:07 GMT, Igor Ignatyev wrote: >> Leonid Mesnik has updated the pull request incrementally with one additional >> commit since the last revision: >> >> copyrights and ident fixed > > test/hotspot/jtreg/vmTestbase/nsk/jvmti/Breakpoint/breakpoint001/breakpoint001.cpp >

Re: RFR: 8259799: vmTestbase/nsk/jvmti/Breakpoint/breakpoint001 is incorrect [v2]

2021-01-14 Thread Leonid Mesnik
> est vmTestbase/nsk/jvmti/Breakpoint/breakpoint001 has incorrect check of > strcmp results here: > > for (i=0; i if (strcmp(methNam,METHODS[i][0]) && > strcmp(methSig,METHODS[i][1])) { > printf("CHECK PASSED: method name: "%s"\tsignature: "%s" %d\n", >