RFR(S) : 8038186 : [TESTBUG] improvements of test j.l.i.MethodHandles
christian.thalinger at oracle.com
Wed Mar 26 20:38:46 UTC 2014
On Mar 24, 2014, at 1:33 PM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
> Hi all,
> Please review the patch:
> - MethodHandlesTest::testCatchException() doesn't provide enough testing of j.l.i.MethodHandles::catchException().
> - MethodHandlesTest contains more than 3k lines, an auxiliary code together w/ a test code, many methods aren't connected w/ each other, etc. All this leads to that it is very very hard to understand, maintain.
Yes, it is.
> - the auxiliary code was moved to testlibray
> - the test code was moved to a separate subpackage
This is very nice. Are there plans to do this for the other existing tests as well?
> - random signature is used for target and handler
> - added scenarios w/ different return types
> webrev: http://cr.openjdk.java.net/~iignatyev/8038186/webrev.01/
> jbs: https://bugs.openjdk.java.net/browse/JDK-8038186
> mlvm-dev mailing list
> mlvm-dev at openjdk.java.net
More information about the core-libs-dev