On Wed, 9 Mar 2022 11:55:27 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
> Here we don't need to check whether it's been acquired because we know it's > been acquired if we reach here. Hello Daniel, may be I am misreading the diff but from what I can see, the `acquire()` method now returns a `boolean` which we are assigning to the local `acquire`. But we aren't checking it for `true` for doing the subsequent operation. So when it reaches the finally block here, as far as I can see, it's not guaranteed that we did indeed `acquire` it. ------------- PR: https://git.openjdk.java.net/jdk/pull/7196