On Fri, 24 Jan 2025 20:13:29 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/runtime/Threads.java >> line 168: >> >>> 166: JvmtiAgentThread, NotificationThread, MonitorDeflationThread, >>> 167: StringDedupThread, AttachListenerThread, >>> DeoptimizeObjectsALotThread and >>> 168: ServiceThread. The latter seven subclasses of the former. Most >>> operations >> >> The latter eight are subclasses of the former? 8-) Or just "JavaThread, or >> its subclasses..." > > Yes, it should be updated to eight, but I think what would be better is > "returns objects of type JavaThraed or one of its subclasses..." Ok, it's fixed now. I stuck with a somewhat minimal update mostly just to resolve the incorrect count. This comment may have further cleanup as part of [JDK-8348347](https://bugs.openjdk.org/browse/JDK-8348347). ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23279#discussion_r1929190501