RFR: 8330534: Update nsk/jdwp tests to use driver instead of othervm

Serguei Spitsyn sspitsyn at openjdk.org
Sat May 4 10:06:54 UTC 2024


On Mon, 22 Apr 2024 11:21:15 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

>> The jdwp tests use debugger and debugee. There is no goal to execute debugger part with all VM flags, they are needed to be used with debugee VM only.
>> The change is all tests is to don't use System.exit() and use 'driver' instead of othervm.
>> test/hotspot/jtreg/vmTestbase/nsk/share/jpda/DebugeeBinder.java
>> is updated to correctly set classpath for debugee
>
> test/hotspot/jtreg/vmTestbase/nsk/jdwp/ThreadReference/Interrupt/interrupt001.java line 93:
> 
>> 91:      */
>> 92:     public static void main (String argv[]) {
>> 93:         int result = run(argv,System.out);
> 
> Suggestion:
> 
>         int result = run(argv, System.out);

Yes, this missing space is all over this fix. Need to fix this formatting.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/18826#discussion_r1589950014


More information about the serviceability-dev mailing list