On Wed, 28 Aug 2024 16:38:06 GMT, Brian Burkhalter <[email protected]> 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