On Mon, 19 Dec 2022 11:26:25 GMT, Romain Manni-Bucau <d...@openjdk.org> 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 other case which should be a better > compromise in practise. > > Another side but nice effect is to have some lighter classloading (subgraph) This was brought up on the core-libs dev mailing list and a JBS issue has been created for this a while back https://bugs.openjdk.org/browse/JDK-8305947. Please update the title of this PR to `8305947: SequenceInputStream implementation can use an Iterator rather than Enumeration` to trigger a official review. ------------- PR Comment: https://git.openjdk.org/jdk/pull/11718#issuecomment-1584033379