On Fri, 24 Jan 2025 11:52:04 GMT, Serguei Spitsyn <sspit...@openjdk.org> 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

Reply via email to