RFR: 8345432: (ch, fs) Replace anonymous Thread with InnocuousThread [v3]

Brian Burkhalter bpb at openjdk.org
Tue Dec 17 16:52:37 UTC 2024


On Tue, 17 Dec 2024 16:27:50 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

>> src/java.base/share/classes/jdk/internal/misc/InnocuousThread.java line 64:
>> 
>>> 62:         return createThread(newName(), target, 0L,
>>> 63:                 ClassLoader.getSystemClassLoader(), -1, true);
>>> 64:     }
>> 
>> I think this is just a bit weird to have this in InnocuousThread. I think we should look for alternatives for PipeImpl so leave it out of this PR as it's benign if it inherits.
>
> I don't like it either. I will revert this commit.

Should the Thread in PipeImpl have a name? Currently it does not.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22697#discussion_r1888875072


More information about the nio-dev mailing list