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

Romain Manni-Bucau duke at openjdk.org
Fri Jun 9 07:27:09 UTC 2023


On Mon, 19 Dec 2022 11:26:25 GMT, Romain Manni-Bucau <duke at 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)

Still hope it can make it to the jdk.

think it is still relevant

would be great to get it onboard instead of letting it dying slowly if possible, guess commit message can be edited at merge time with github and comments were addressed AFAIK so looks ready for me

-------------

PR Comment: https://git.openjdk.org/jdk/pull/11718#issuecomment-1384464985
PR Comment: https://git.openjdk.org/jdk/pull/11718#issuecomment-1438146415
PR Comment: https://git.openjdk.org/jdk/pull/11718#issuecomment-1544428157


More information about the core-libs-dev mailing list