On 21/02/2019 05:19, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8219139/webrev.00/index.html
40 lines changed: 17 ins; 2 del; 21 mod;
Hi all,

could you please review this small patch which moves tests from test/jdk/vm?

there are 16 tests in test/jdk/vm directory. all but JniInvocationTest are 
hotspot tests, so they are moved to test/hotspot test suite; JniInvocationTest 
is a launcher test and hence it's moved to test/jdk/tools/launcher. as 
hotspot/compiler and hotspot/gc tests use packages, the tests moved there have 
been updated to have a package statement.

webrev: http://cr.openjdk.java.net/~iignatyev//8219139/webrev.00/index.html
JBS: https://bugs.openjdk.java.net/browse/JDK-8219139
testing: tier[1-2] (in progress), all the touched tests locally

The tools/launcher directory is the tests for the java launcher, I don't think it's the right place for tests for the JNI invocation interface - I think that one is asking to be moved to be with the other JNI tests.

-Alan

Reply via email to