Looks like the line "# } nsk/jvmti" is a left over. Otherwise this looks
ok, even if it's an enormous amount of duplication. Hopefully we can
figure out a better way to express common parameters for tests soon.
/Erik
On 2018-05-22 16:35, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8199383/webrev.00/index.html
308253 lines changed: 308253 ins; 0 del; 0 mod;
Hi all,
could you please review this patch which open sources JVMTI tests from VM
testbase?
As usually w/ VM testbase code, these tests are old, they have been run in
hotspot testing for a long period of time. Originally, these tests were run by
a test harness different from jtreg and had different build and execution
schemes, some parts couldn't be easily translated to jtreg, so tests might have
actions or pieces of code which look weird. In a long term, we are planning to
rework them.
webrev: http://cr.openjdk.java.net/~iignatyev//8199383/webrev.00/index.html
JBS: https://bugs.openjdk.java.net/browse/JDK-8199383
testing: :vmTestbase_nsk_jvmti test group
Thanks,
-- Igor