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

Reply via email to