On Wed, 21 Jun 2023 15:03:12 GMT, Sergey Tsypanov <[email protected]> wrote:
>> Just a tiny clean-up to remove racy read within synchronized method > > Sergey Tsypanov has updated the pull request incrementally with one > additional commit since the last revision: > > Update src/java.base/share/classes/java/io/PipedOutputStream.java > > Co-authored-by: liach <[email protected]> I think the bug description should include a bit more detail along the lines that since the method reading and writing are synchronized, its not really a racy read. It does avoid reading the volatile value twice (a relatively expensive operation). ------------- PR Comment: https://git.openjdk.org/jdk/pull/14589#issuecomment-1603308404
