On Mon, 22 Apr 2024 11:21:15 GMT, Andrey Turbanov <[email protected]> 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
