On Tue, 18 Jan 2022 15:57:58 GMT, Roger Riggs <[email protected]> wrote:
> The thread factory used to create the process reaper threads unnecessarily
> inherits the callers thread context classloader.
> The result is retention of the class loader.
>
> The thread factory used for the pool of process reaper threads is modified to
> use an InnocuousThread with a given stacksize.
> The test verifies that the process reaper threads have a null context
> classloader.
src/java.base/share/classes/jdk/internal/misc/InnocuousThread.java line 138:
> 136: Thread t = new InnocuousThread(INNOCUOUSTHREADGROUP,
> 137: target, name, stackSize, loader);
> 138: if (priority >= 0) {
Adding an addition newSystemThread method should be okay but can you fix the
formatting/line lengths so it's a bit more inconsistent with the existing code.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7131
