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