RFR (M) 8009577: Test test/closed/runtime/classunload broken

[Mikael Gerdin]

Hi Christian,

I'm not a Reviewer, but I think the tests look good.

/Mikael

On 04/24/2013 07:46 PM, Christian Tornqvist wrote:
> Hi everyone,
>
> Still looking for a review on this.
>
> Thanks,
>
> Christian
>
> *From:*hotspot-runtime-dev-bounces at openjdk.java.net
> [mailto:hotspot-runtime-dev-bounces at openjdk.java.net] *On Behalf Of
> *Christian Tornqvist
> *Sent:* den 4 april 2013 03:08
> *To:* hotspot-runtime-dev at openjdk.java.net
> *Subject:* RE: RFR (M) 8009577: Test test/closed/runtime/classunload broken
>
> Hi everyone,
>
> Still looking for more reviews on this.
>
> Thanks,
>
> Christian
>
> *From:*hotspot-runtime-dev-bounces at openjdk.java.net
> <mailto:hotspot-runtime-dev-bounces at openjdk.java.net>
> [mailto:hotspot-runtime-dev-bounces at openjdk.java.net] *On Behalf Of
> *christian.tornqvist at oracle.com <mailto:christian.tornqvist at oracle.com>
> *Sent:* den 18 mars 2013 15:33
> *To:* hotspot-runtime-dev at openjdk.java.net
> <mailto:hotspot-runtime-dev at openjdk.java.net>
> *Subject:* RFR (M) 8009577: Test test/closed/runtime/classunload broken
>
> Hi everyone,
>
> This is fix and a relocation of two tests in runtime/ClassUnload. First
> fix was to make them compatible with the new WB API changes that went in
> a month ago, second fix was to let the custom classloader load the class
> from ./classes. These tests were not in the public before but I see no
> reason for not moving them as part of this change. I also split
> KeepAlive.java into 4 different java files to make it more readable and
> easier to debug.
>
> Bug (not yet visible):
>
> http://bugs.sun.com/view_bug.do?bug_id=8009577
>
> Webrev:
>
> http://cr.openjdk.java.net/~ctornqvi/webrev/8009577/webrev.00/
>
> Thanks,
>
> Christian
>