RFR: 8361325: Refactor ClassLoaderExt
Serguei Spitsyn
sspitsyn at openjdk.org
Thu Jul 3 15:13:39 UTC 2025
On Thu, 3 Jul 2025 05:16:54 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:
> After the refactoring work done via [JDK-8280682](https://bugs.openjdk.org/browse/JDK-8280682), there are only three functions remaining in the ClassLoaderExt class. This RFE is to move those remaining functions into other classes so that the ClassLoaderExt class can be eliminated.
>
> Testing: passed tiers 1 - 5.
It looks good to me. I like this simplification.
But I'm curious what was the original reason to have the `classLoaderExt`?
src/hotspot/share/cds/classListParser.cpp line 33:
> 31: #include "cds/metaspaceShared.hpp"
> 32: #include "cds/unregisteredClasses.hpp"
> 33: #include "classfile/classLoader.hpp"
Nit: I wonder if the line #33 is really needed and can be removed.
-------------
Marked as reviewed by sspitsyn (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/26110#pullrequestreview-2983562923
PR Review Comment: https://git.openjdk.org/jdk/pull/26110#discussion_r2183031189
More information about the serviceability-dev
mailing list