XS review: 8098557: Eager initialization of a super interface with a default method

[harold seigel]

Hi Karen,

Your changes look good, but I didn't quite understand this new comment 
in instanceKlass.cpp:

+ // If C is an interface that declares a non-abstract, non-static method,
+ // the initialization of a class (not an interface) that implements C 
directly or
+ // indirectly.

Could you re-phrase it? (No need to see a new webrev.)

Thanks, Harold


On 11/20/2015 2:29 PM, Karen Kinnear wrote:
> Please review fix for 8098557:
> Summary: Initialization of a super interface with a default method should
> be triggered by initialization of a dependent subclass, but not of a
> dependent subinterface.
>
> open webrev at http://cr.openjdk.java.net/~acorn/8098557/webrev
> bug link at https://bugs.openjdk.java.net/browse/JDK-8098557
>
> tests:
> jprt
> reported example, new test
> all platforms: vm quick & non-cool
> linux-x64: defmeth, SelectionResolution, jtreg hotspot, hotspot/closed, compiler, compiler/closed
>             jtreg java/util, jtreg tools/javac/lambdaShapes
> jtreg jdk in progress
> jcks  - working on
>
> thanks,
> Karen