On Wed, 21 Jun 2023 15:03:12 GMT, Sergey Tsypanov <stsypa...@openjdk.org> 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 <7806504+li...@users.noreply.github.com> 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
