Re: RFR: 8354724: Methods in java.io.Reader to read all characters and all lines [v18]

2025-05-13 Thread Brian Burkhalter
On Tue, 13 May 2025 10:55:50 GMT, Alan Bateman wrote: >> 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 >> commi

Re: RFR: 8354724: Methods in java.io.Reader to read all characters and all lines [v18]

2025-05-13 Thread Brian Burkhalter
On Tue, 13 May 2025 15:25:57 GMT, Brian Burkhalter wrote: >> src/java.base/share/classes/java/io/Reader.java line 430: >> >>> 428: * The behavior for the case where the reader is >>> 429: * asynchronously closed, or the thread interrupted during >>> the >>> 430: * read, is highl

Re: RFR: 8354724: Methods in java.io.Reader to read all characters and all lines [v18]

2025-05-13 Thread Brian Burkhalter
On Tue, 13 May 2025 14:47:53 GMT, Roger Riggs wrote: > I think this (with fix above for stream -> reader) is ready to create the CSR > ... Will do today. Thanks. - PR Comment: https://git.openjdk.org/jdk/pull/24728#issuecomment-2876983943

Re: RFR: 8354724: Methods in java.io.Reader to read all characters and all lines [v18]

2025-05-13 Thread Roger Riggs
On Mon, 12 May 2025 23:51:19 GMT, Brian Burkhalter 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 m

Re: RFR: 8354724: Methods in java.io.Reader to read all characters and all lines [v18]

2025-05-13 Thread Alan Bateman
On Mon, 12 May 2025 23:51:19 GMT, Brian Burkhalter 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 m

Re: RFR: 8354724: Methods in java.io.Reader to read all characters and all lines [v18]

2025-05-13 Thread Alan Bateman
On Mon, 12 May 2025 23:51:19 GMT, Brian Burkhalter 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 m

Re: RFR: 8354724: Methods in java.io.Reader to read all characters and all lines [v18]

2025-05-12 Thread Brian Burkhalter
> 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