Integrated: 8266078: Reader.read(CharBuffer) advances Reader position for read-only Charbuffers

Brian Burkhalter bpb at openjdk.java.net
Thu Apr 29 15:58:03 UTC 2021


On Tue, 27 Apr 2021 18:47:41 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.

This pull request has now been integrated.

Changeset: 5f156660
Author:    Brian Burkhalter <bpb at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/5f15666092d9928c07cbe66cdd96538459ffe393
Stats:     71 lines in 2 files changed: 71 ins; 0 del; 0 mod

8266078: Reader.read(CharBuffer) advances Reader position for read-only Charbuffers

Reviewed-by: rriggs, alanb, chegar

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

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


More information about the core-libs-dev mailing list