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

Markus KARG duke at openjdk.org
Mon Nov 21 17:43:23 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:

  private member variables

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/11249/files
  - new: https://git.openjdk.org/jdk/pull/11249/files/2e957354..8d298318

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

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 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