On Fri, 18 Sep 2020 20:01:02 GMT, Daniel D. Daugherty <[email protected]>
wrote:
>> Robbin Ehn has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fixed double checks
>> Added NSV
>> ProcessResult to enum
>> Fixed logging
>> Moved _active_handshaker to private
>
> src/hotspot/share/prims/jvmtiEventController.cpp line 340:
>
>> 338: } else {
>> 339: Handshake::execute(&hs, target);
>> 340: }
>
> This guarantee() that the handshake has executed doesn't have an
> equivalent in the rewritten code. Should there be some way of verifying
> this condition from this location?
Fixed
-------------
PR: https://git.openjdk.java.net/jdk/pull/151
