JDK-8027232 : Update j.l.invoke code generating class files to use ASM enhancements for invocation of non-abstract methods on ifaces
  • Type: Bug
  • Component: core-libs
  • Sub-Component: java.lang.invoke
  • Affected Version: 8
  • Priority: P3
  • Status: Closed
  • Resolution: Fixed
  • Submitted: 2013-10-24
  • Updated: 2013-11-22
  • Resolved: 2013-11-06
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 8 Other
8 b117Fixed port-stage-ppc-aixFixed
Related Reports
Blocks :  
When JDK-8027227 is fixed the files InnerClassLambdaMetafactory.java and InvokerBytecodeGenerator.java need to be updated for calls to MethodVistor.visitMethodInsn when the method to be invoked is a non-abstract method on an interface.

In addition the major version of the generated class files should be bumped up to 52 (Opcodes.V1_8).
8-critical-sqe: agreed this should be fixed, no test dev necessary (as we only test ASM through testing lambda).

Thanks for the pointer to the webrev. Agree to get these changes pushed.

The webrev for this is here: http://cr.openjdk.java.net/~ksrini/8027232/webrev.1/ This has a unit test as described in the changeset.

Two questions: - Where can I get the changeset for this fix? - Is there a unit test needed for this fix or testing covered by existing regression tests, same as JDK-8027227? Maybe a noreg-* label is needed for this bug as well?

8-critical-dev: This fix is absolutely required for DefaultMethods to work correctly and for JSR-335 compliance, otherwise we will encounter regressions and hard to detect subtle issues, note that issue JDK-8027227, also needs to be approved.

Dependent bug JDK-8027227 isn't going to be fixed by ZBB.