[lworld] RFR: 8244231: [lworld] Add support for ref-default and val-default inline classes. [v3]

Srikanth Adayapalam sadayapalam at openjdk.java.net
Fri Jul 23 02:09:28 UTC 2021

On Fri, 23 Jul 2021 01:27:12 GMT, Srikanth Adayapalam <sadayapalam at openjdk.org> wrote:

>> Code changes and tests
> Srikanth Adayapalam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains ten additional commits since the last revision:
>  - Merge branch 'lworld' into JDK-8244231
>  - Merge branch 'lworld' into JDK-8244231
>  - Fix confusingi/unhelpful javadoc per review comments
>  - Adjust tests post JDK-8237073
>  - Merge branch 'lworld' into JDK-8244231
>  - Fix white space problems
>  - Fix white space problems
>  - Modify several existing tests to verify behavior with reference default classes
>  - 8244231: Add support for ref-default and val-default inline classes.

Thanks Maurizio!

The confusing and unhelpful javadoc issues have been fixed.

In JDK-8268734 ("Various internal Type/Symbol APIs need rationalization/adjustment") I will follow up with a comprehensive review of all the Type and Symbol APIs - these APIs were rolled up as and when needed at widely different points in time 
and it is an useful exercise to take a step back and review them in a holistic fashion.

In JDK-8270882 (" Loose ends in reference default primitive class implementation.") I will follow up on the various known
issues that have been deliberately deferred to a later pass so as to make it more easily manageable,


PR: https://git.openjdk.java.net/valhalla/pull/482

More information about the valhalla-dev mailing list