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

2023-06-09 Thread Romain Manni-Bucau
On Tue, 11 Apr 2023 07:48:45 GMT, Jens Lidestrom 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-06-09 Thread Romain Manni-Bucau
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 iter

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

2023-06-09 Thread Romain Manni-Bucau
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 other case which should be a better compromise in practise. Another s