On Mon, 24 May 2021 16:28:15 GMT, Daniel D. Daugherty <dcu...@openjdk.org> wrote:
>> Robbin Ehn has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Final fixes: last famous words > > src/hotspot/share/prims/jvmtiRawMonitor.cpp line 405: > >> 403: >> 404: guarantee(self == _owner, "invariant"); >> 405: guarantee(save == _recursions, "invariant"); > > Since you just restored the `_recursions` field on L402, why do > you need this guarantee() 2 code lines later? I left it since if we do changes here those two should be true even after any changes. I can remove it. ------------- PR: https://git.openjdk.java.net/jdk/pull/3875
