> 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: Alan's Proposal ------------- Changes: - all: https://git.openjdk.org/jdk/pull/11249/files - new: https://git.openjdk.org/jdk/pull/11249/files/b6ccec20..9c902512 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=11249&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=11249&range=01-02 Stats: 6 lines in 1 file changed: 4 ins; 1 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