Re: RFR: 8305947: SequenceInputStream implementation can use an Iterator rather than Enumeration

2023-07-24 Thread Benjamin Marwell
On Mon, 19 Dec 2022 11:26:25 GMT, Romain Manni-Bucau wrote: > enumeration(list) will create an enumeration, a list and an iterator whereas > the impl only requires an iterator > this PR drops the enumeration wrapper for binary constructor and just maps > the enumeration to an iterator for the o

Re: RFR: 8305947: SequenceInputStream implementation can use an Iterator rather than Enumeration

2023-07-12 Thread Benjamin Marwell
On Fri, 9 Jun 2023 15:49:03 GMT, Brian Burkhalter wrote: > Please see [I have a patch, what do I > do?](https://openjdk.org/guide/#i-have-a-patch-what-do-i-do) and in > particular [Socialize your > change](https://openjdk.org/guide/#socialize-your-change). Thanks, will put it on the mailing l

Re: RFR: 8305947: SequenceInputStream implementation can use an Iterator rather than Enumeration

2023-06-09 Thread Benjamin Marwell
On Tue, 24 Jan 2023 06:54:19 GMT, Jaikiran Pai wrote: > PRs which do not have a corresponding bug tracker issue Will write to the mailing list - PR Comment: https://git.openjdk.org/jdk/pull/11718#issuecomment-1464897988

Re: RFR: 8305947: SequenceInputStream implementation can use an Iterator rather than Enumeration

2023-06-09 Thread Benjamin Marwell
On Mon, 19 Dec 2022 11:26:25 GMT, Romain Manni-Bucau wrote: > enumeration(list) will create an enumeration, a list and an iterator whereas > the impl only requires an iterator > this PR drops the enumeration wrapper for binary constructor and just maps > the enumeration to an iterator for the o

Re: RFE: SequenceInputStream - Consider new issue for PR #11718

2023-04-11 Thread Benjamin Marwell
Follow up, I would highly appreciate it if anyone would be so kind as to open an issue for this enhancement. Am Sa., 8. Apr. 2023 um 17:36 Uhr schrieb Benjamin Marwell : > > Dear Core-Devs, > > There was a PR created for an enhancement of SequenceInputStream [1] > by Roma

RFE: SequenceInputStream - Consider new issue for PR #11718

2023-04-08 Thread Benjamin Marwell
Dear Core-Devs, There was a PR created for an enhancement of SequenceInputStream [1] by Romain Manni-Buceau. Description by Romain: > enumeration(list) will create an enumeration, a list and an iterator whereas > the impl only requires an iterator > this PR drops the enumeration wrapper for bin