On Mon, 26 May 2025 16:06:01 GMT, Andrey Turbanov <aturba...@openjdk.org> wrote:

>> Alan Bateman has updated the pull request with a new target base due to a 
>> merge or a rebase. The incremental webrev excludes the unrelated changes 
>> brought in by the merge/rebase.
>
> src/java.base/share/classes/jdk/internal/vm/ThreadDumper.java line 186:
> 
>> 184:         Thread.State state = snapshot.threadState();
>> 185:         writer.println("#" + thread.threadId() + " \"" + 
>> snapshot.threadName()
>> 186:                 +  "\" " + (thread.isVirtual() ? "virtual " : "") + 
>> state + " " + now);
> 
> Suggestion:
> 
>                 + "" " + (thread.isVirtual() ? "virtual " : "") + state + " " 
> + now);

okay

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25429#discussion_r2111404652

Reply via email to