RFR: 8348240: Remove SystemDictionaryShared::lookup_super_for_unregistered_class() [v2]

Ioi Lam iklam at openjdk.org
Thu Jan 23 22:05:59 UTC 2025


On Wed, 22 Jan 2025 23:41:18 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Ioi Lam 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 three additional commits since the last revision:
>> 
>>  - Merge branch 'master' into 8348240-remove-lookup_super_for_unregistered_class
>>  - @calvinccheung comments
>>  - 8348240: Remove SystemDictionaryShared::lookup_super_for_unregistered_class()
>
> I like this change a lot, except for the name "unregistered" but that can be taken up in some other way.  If I understand correctly, this just creates a dummy class loader to load the classes you want to share from the non-boot, app or system class loader.

Thanks @coleenp @calvinccheung for the review.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/23226#issuecomment-2611107641


More information about the core-libs-dev mailing list