On Fri, 24 Jan 2025 11:52:04 GMT, Serguei Spitsyn <[email protected]> wrote:
>> Chris Plummer has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Some minor cleanup.
>
> src/jdk.jdwp.agent/share/native/libjdwp/stepControl.c line 913:
>
>> 911:
>> 912: if (needsSuspending) {
>> 913: tty_message("clearStep: suspending thread");
>
> Nit: This `tty_message()` is still here. Should it be at least under `#ifdef
> DEBUG`?
It's gone. Are you sure you are looking at the latest sources?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23182#discussion_r1929176082