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