On Fri, 24 Mar 2023 06:31:14 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:

>> The TestScaffold incorrectly parse options, it should insert wrapper class 
>> between VM options and applications classame.
>
> Leonid Mesnik has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   added comments and trim arguments

test/jdk/com/sun/jdi/TestScaffold.java line 469:

> 467:         // the first argument not-starting with '-' is treated as a 
> classname
> 468:         // the other arguments are split to targetVMArgs 
> targetAppCommandLine correspondingly
> 469:         // The example of args for line '@run driver Frames2Test -Xss4M' 
> is  '-Xss4M' 'Frames2Targ'.

Is `Frames2Test` a typo there?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13170#discussion_r1148791133

Reply via email to