On Fri, 28 Aug 2026 15:35:27 GMT, Alan Bateman <[email protected]> wrote:

> > Pulling the `readCalled = true;` line up into `read()` as suggested by Alan 
> > implied adding `synchronized(this)` in `read()
> 
> Maybe I should have been clear, I meant the 3-arg read method, and the read0 
> method. They are the "front door" and already have the synchronized block.

I see. Thank you for the clarification. I removed the changes from `read()` and 
`implRead()` again. In this case, `read0()` must stay `synchronized`, certainly.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32264#discussion_r3882123624

Reply via email to