RFR: 8354724: BufferedReader readAllLines and readString methods

[Chen Liang]

On Thu, 17 Apr 2025 16:18:55 GMT, Brian Burkhalter <bpb at 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