[lworld] RFR: 8335545: [lworld] JNI MonitorEnter doesn't throw the right exception [v2]

Frederic Parain fparain at openjdk.org
Wed Jul 10 20:28:56 UTC 2024


On Tue, 9 Jul 2024 21:18:38 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Frederic Parain has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Changes suggested by Coleen
>
> src/hotspot/share/runtime/synchronizer.cpp line 564:
> 
>> 562: void ObjectSynchronizer::enter(Handle obj, BasicLock* lock, JavaThread* current) {
>> 563:   assert(current == Thread::current(), "must be");
>> 564:   JavaThread* THREAD = current;
> 
> You don't need the THREAD line anymore.

Line removed.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1160#discussion_r1672085414


More information about the valhalla-dev mailing list