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