RFR: 8266078: Reader.read(CharBuffer) advances Reader position for read-only Charbuffers [v3]

Chris Hegarty chegar at openjdk.java.net
Wed Apr 28 15:40:54 UTC 2021

On Wed, 28 Apr 2021 15:29:20 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

>> Please consider this request to modify `Reader.read(CharBuffer)` to check whether the buffer is read-only before reading any characters from the character stream. This can happen now if the buffer is read-only. Character are first read thereby advancing the stream before an attempt is made to put them in the `CharBuffer` thus incorrectly advancing the stream position.
> Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:
>   8266078: Remove confusing comment

Marked as reviewed by chegar (Reviewer).


PR: https://git.openjdk.java.net/jdk/pull/3725

More information about the core-libs-dev mailing list