On Thu, 17 Apr 2025 16:18:55 GMT, Brian Burkhalter <b...@openjdk.org> wrote:
> Implement the requested methods and add a test thereof.
Is there a particular reason that these methods should exist only on
`BufferedReader` but not the base `Reader`? Similar read-all operation
`InputStream::readAllBytes` exists on the base `InputStream` class.
src/java.base/share/classes/java/io/BufferedReader.java line 419:
> 417: *
> 418: * @apiNote
> 419: * <p> This method is intended for simple cases where it is
> convenient
Suggestion:
* This method is intended for simple cases where it is convenient
API notes already have a dd/dt block, no need for extra paragraphs.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24728#issuecomment-2813602189
PR Review Comment: https://git.openjdk.org/jdk/pull/24728#discussion_r2049374273
