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
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)
-------------
Commit messages:
- [JDK-8305947] Unify SequenceInputStream 'backend' using an Iterator which enables to avoid one abstraction for binary case
Changes: https://git.openjdk.org/jdk/pull/11718/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=11718&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8305947
Stats: 19 lines in 1 file changed: 12 ins; 2 del; 5 mod
Patch: https://git.openjdk.org/jdk/pull/11718.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/11718/head:pull/11718
PR: https://git.openjdk.org/jdk/pull/11718
More information about the core-libs-dev
mailing list