On Wed, 28 Aug 2024 14:53:37 GMT, Jaikiran Pai <[email protected]> wrote:
>> Brian Burkhalter has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8336895: Modified verbiage per reviewer comment
>
> 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.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20320#discussion_r1734992217
