On Thu, 22 Jan 2026 02:22:23 GMT, Chris Plummer <[email protected]> wrote:

>> Yasumasa Suenaga has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Update TestJhsdbJstackMixed.java
>
> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/debugger/windbg/WindbgCDebugger.java
>  line 88:
> 
>> 86:                                         .filter(jt -> 
>> thread.equals(jt.getThreadProxy()))
>> 87:                                         .findFirst()
>> 88:                                         .orElse(null);
> 
> I wonder if this code should be moved into its own API. Call it something 
> like mapThreadProxyToJavaThread(), or maybe getJavaThreadForThreadProxy(). It 
> would make the code more readable.

Thanks for your comment. I extracted this logic as 
`mapThreadProxyToJavaThread()`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29019#discussion_r2719408169

Reply via email to