[8u-dev] Request for approval: 8163969: Cyclic interface initialization causes JVM crash

[Coleen Phillimore]

Thanks George.

On 10/12/16 7:25 PM, George Triantafillou wrote:
> Hi Coleen,
>
> Small typo in src/share/vm/oops/instanceKlass.cpp:
>
> 889   // Next, if C is a class rather than an interface, initialize 
> it's super class and super
>
> change to
>
> 889   // Next, if C is a class rather than an interface, initialize 
> its super class and super
>
> Otherwise, looks good.

I always want to type the ' in it's for some reason.  I fixed it.

Coleen

>
> -George
>
> On 10/12/2016 5:10 PM, Coleen Phillimore wrote:
>> Summary: Backport change to correct interface initialization.
>>
>> There were too many changes to instanceKlass.cpp for a clean 
>> backport.  Also in JDK8, this corrects interface initialization to 
>> not initialize the whole interface hierarchy if an interface, not 
>> class, initializes initialization.  This is to correctly follow JLS 
>> 12.4.2 step 7.   I filed a compatibility request (in review) to 
>> document the difference in behavior, which I believe will not be 
>> noticed.
>>
>> Tested with JPRT, including runtime jtreg lambda-features tests, and 
>> JCK tests.
>>
>> open webrev at http://cr.openjdk.java.net/~coleenp/8163969.8.01/webrev
>> bug link https://bugs.openjdk.java.net/browse/JDK-8163969.8
>>
>> Thanks,
>> Coleen
>