[11u] RFR: 8224555: vmTestbase/nsk/jvmti/scenarios/contention/TC02/tc02t001/TestDescription.java failed

[Doerr, Martin]

Hi Götz,

you have additional leading whitespaces in tc02t001.c:
  static jint lines[] = { 48, 53, 58 };

Looks good otherwise. Don't need to see another webrev.

Best regards,
Martin


> -----Original Message-----
> From: jdk-updates-dev <jdk-updates-dev-retn at openjdk.java.net> On
> Behalf Of Lindenmaier, Goetz
> Sent: Dienstag, 22. September 2020 08:56
> To: jdk-updates-dev at openjdk.java.net
> Subject: [11u] RFR: 8224555:
> vmTestbase/nsk/jvmti/scenarios/contention/TC02/tc02t001/TestDescription
> .java failed
> 
> Hi,
> 
> I am downporting this for parity with 11.0.10-oracle.
> 
> I had to resolve because one file is C and not C++.
> Also, in tc02t001.java, the deleted sleep() times differ.
> 
> (The patch looks complicated but is not. The change switches
> position of class tc02t001Thread and class tc02t001,
> and webrev computed a different diff than to be seen in
> the repo.)
> 
> http://cr.openjdk.java.net/~goetz/wr20/8224555-nsk_jvmti_test-jdk11/01/
> 
> Please review.
> 
> https://bugs.openjdk.java.net/browse/JDK-8224555
> http://hg.openjdk.java.net/jdk/jdk13/rev/e17c9a93b505
> 
> Best regards,
>   Goetz.