RFR: 8345543: Test serviceability/jvmti/vthread/StopThreadTest/StopThreadTest.java failed: expected JVMTI_ERROR_OPAQUE_FRAME instead of: 0 [v2]

Serguei Spitsyn sspitsyn at openjdk.org
Wed Jan 15 19:29:08 UTC 2025


On Wed, 15 Jan 2025 16:14:26 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review: nit: move curly bracket down
>
> src/hotspot/share/runtime/objectMonitor.cpp line 495:
> 
>> 493:   freeze_result result;
>> 494: 
>> 495:   {
> 
> I think we should move this curly brace down too.

I did not want to deal with the re-indent so it is easy to review.
But you are right I've moved the curly brace down now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23125#discussion_r1917226574


More information about the hotspot-runtime-dev mailing list