RFR: 8336895: BufferedReader doesn't read full \r\n line ending when it doesn't fit in buffer [v3]

[Brian Burkhalter]

On Wed, 28 Aug 2024 16:38:06 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

>> src/java.base/share/classes/java/io/BufferedReader.java line 62:
>> 
>>> 60:  * replacing each DataInputStream with an appropriate BufferedReader.
>>> 61:  *
>>> 62:  * <p> More than one instance of BufferedReader should not be used with the
>> 
>> Nit - In the `BufferedInputStream` and the `BufferedOutputStream` we use `{@code ...}` to refer to these classes. Perhaps we should do the same here for `BufferedReader` and `Reader`? Same comment in the `BufferedWriter`  class update.
>
> Yes, I saw that but did not want to change it in the initial pass(es) as it's too much noise. If things are otherwise quiescent, then I can change it.

Fixed in d6831f9 and added a link for `line.separator`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20320#discussion_r1735090929