RFR: 8306308: (ch) Writer created by Channels::newWriter may lose data [v5]

Daniel Jeliński djelinski at openjdk.org
Fri Oct 20 06:45:40 UTC 2023


On Thu, 19 Oct 2023 23:46:03 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

>> Change `Channels.newWriter` to return a `Writer` created by first wrapping the `WritableByteChannel` in an `OutputStream` using `Channels.newOutputStream` and then wrapping the result in an `OutputStreamWriter`. With this change in place, the problems reported in the issue do not occur.
>
> Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8306308: Add test

Changes requested by djelinski (Reviewer).

` line 1:

> (failed to retrieve contents of file, check the PR for context)
This filename doesn't look good

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

PR Review: https://git.openjdk.org/jdk/pull/16207#pullrequestreview-1689336359
PR Review Comment: https://git.openjdk.org/jdk/pull/16207#discussion_r1366535724


More information about the nio-dev mailing list