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

Reply via email to