Approved.

regards,
Sean.

On 15/10/14 09:56, Kevin Walls wrote:
Hi,

I'd like to get a approval for this backport into 7u. It's a change from the original fix, approval below from original author.

webrev
http://cr.openjdk.java.net/~kevinw/8058927/webrev.00/
bug
https://bugs.openjdk.java.net/browse/JDK-8058927

JPRT, the testcase shown here, and the original test that found the problem have tested OK.

Thanks
Kevin


------- Original Message --------
Subject:     Re: RFR 8058927: ATG throws ClassNotFoundException
Date:     Tue, 14 Oct 2014 15:35:02 -0400
From:     Coleen Phillimore <coleen.phillim...@oracle.com>
To:     hotspot-runtime-...@openjdk.java.net



Kevin,
The code looks good.
Coleen

On 10/13/14, 4:49 AM, Kevin Walls wrote:
Hi,

This is already in 8u, I'd like to get a review of this backport for
7u, as it isn't quite the same as the original fix.

webrev
http://cr.openjdk.java.net/~kevinw/8058927/webrev.00/

bug
https://bugs.openjdk.java.net/browse/JDK-8058927

I had to put a check for null in objArrayKlass::class_loader(), as it
is called from Universe::fixup_mirrors  at startup time.

JPRT, the testcase shown here and the original test that found the
problem have tested OK.

Thanks
Kevin




Reply via email to