RFR 8058927: ATG throws ClassNotFoundException

[Coleen Phillimore]

Thank you, Dan.

On 9/26/14, 8:59 AM, Daniel D. Daugherty wrote:
> On 9/25/14 8:05 PM, Coleen Phillimore wrote:
>>
>> Thanks for pointing out I cut/pasted the wrong webrevs.  Here they are:
>>
>> open webrev at http://cr.openjdk.java.net/~coleenp/8058927/
>
> src/share/vm/classfile/javaClasses.cpp
>     No comments.
>
> src/share/vm/oops/arrayKlass.cpp
>     No comments.
>
> test/runtime/LoadClass/ShowClassLoader.java
>     Extra space before last ')' on lines 38-39.
>
>     I'm curious why the test shows the classloader for Object[]
>     but doesn't use it for anything else...

It's just what David's test did.  I added the check for null for the 
array with the component that had a non-null class loader.  I guess I 
could assert that "[java/lang/Object" class loader is null but I'm 
afraid modules or somebody might create some primordial class loader in 
the future and break the test.  I guess that would break a lot of things 
though.

Coleen

>
> Thumbs up.
>
> Dan
>
>
>
>
>> bug link https://bugs.openjdk.java.net/browse/JDK-8058927
>>
>> Thanks,
>> Coleen
>>
>> On 9/25/14, 7:33 PM, Coleen Phillimore wrote:
>>> Summary: ClassLoader for array klass was set to null and not the 
>>> class loader of the component type.
>>>
>>> open webrev at http://cr.openjdk.java.net/~coleenp/8058843
>>> bug link https://bugs.openjdk.java.net/browse/JDK-8058843
>>>
>>> See bug for more details.
>>>
>>> Ran testbase, jtreg and jck tests, not that they found this bug in 
>>> the first place.  Boris verified the fix with ATG.
>>>
>>> Thank you to David Holmes for finding it through visual inspection 
>>> of the original change.
>>>
>>> Thanks,
>>> Coleen
>>
>