On Mon, 12 May 2025 23:51:19 GMT, Brian Burkhalter <b...@openjdk.org> wrote:
>> Implement the requested methods and add a test thereof. > > Brian Burkhalter has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains 21 additional > commits since the last revision: > > - 8354724: Attempt to further improve verbiage > - Merge > - Update src/java.base/share/classes/java/io/Reader.java > > 8354724: Apply suggested changes for readAllLines. > > Co-authored-by: Roger Riggs <roger.ri...@oracle.com> > - Update src/java.base/share/classes/java/io/Reader.java > > 8354724: Apply suggested changes for readAllAsString. > > Co-authored-by: Roger Riggs <roger.ri...@oracle.com> > - 8354724: Revert BufferedReader; simplify Reader changes removing > overridable self-call; update test > - 8354724: Fix potential NPE in test > - 8354724: Fix readAllChars gaffe in Reader returned by Readed.of and > account for it in test > - 8354724: Improve test per reviewer comments > - 8354724: Add missing toList(), tweak verbiage; update test > - 8354724: Add Reader.readAllLines; test not yet updated > - ... and 11 more: https://git.openjdk.org/jdk/compare/dcf15854...1d811589 Looks good, I think this (with fix above for stream -> reader) is ready to create the CSR and get that review process moving. ------------- PR Comment: https://git.openjdk.org/jdk/pull/24728#issuecomment-2876832823
