RFR: JDK-8297299 SequenceInputStream should not use Vector [v2]

Markus KARG duke at openjdk.org
Sun Nov 20 09:07:21 UTC 2022


> There is no need to use a temporary Vector within the constructor of SynchronizedInputStream, as more efficient (non-synchronized) alternative code (like List.of) will do the same in possibly less time. While the optimization is not dramatic, it still makes sense to replace Vector unless synchronization is really needed.

Markus KARG has updated the pull request incrementally with one additional commit since the last revision:

  allowing s2 to be null

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/11249/files
  - new: https://git.openjdk.org/jdk/pull/11249/files/3495a8b2..b6ccec20

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=11249&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=11249&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/11249.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/11249/head:pull/11249

PR: https://git.openjdk.org/jdk/pull/11249


More information about the core-libs-dev mailing list