On Wed, 30 Sep 2020 18:12:06 GMT, Igor Ignatyev <iignat...@openjdk.org> wrote:
>> I guess a better question would be is where is ~ being used in the arguments? > > not in the tests which use `nsk/share/jvmti/ArgumentHandler`, otherwise, we > would see these tests failing as wouldn't > parse them correctly. quick grepping showed that that > * `~` is used in `vm/mlvm/*/jvmti` tests, but these tests don't use > `nsk/share/jvmti/ArgumentHandler`, they use > `nsk/share/ArgumentParser/IgnoreUnknownArgumentParser` which doesn't parse > agent options string at all > * `,` has been used in `nsk/jvmti/scenarios/contention/TC05/tc05t001/` > before this patch, and the test wasn't getting > options right, it was setting `verbose,waittime` equal to `5`, instead of > `verbose` to an empty string and `waittime` > to `5`. Ok. ------------- PR: https://git.openjdk.java.net/jdk/pull/370