> Usage of thread-safe collection `Vector` is unnecessary. It's recommended to 
> use `ArrayList` if a thread-safe implementation is not needed. In 
> post-BiasedLocking times, this is gets worse, as every access is synchronized.
> I checked only places where `Vector` was used as local variable.

Andrey Turbanov has updated the pull request incrementally with one additional 
commit since the last revision:

  8271603: Unnecessary Vector usage in java.desktop
  use toArray with zero sized array in places where Vector.copyInto was used

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/4680/files
  - new: https://git.openjdk.java.net/jdk/pull/4680/files/5727f6f2..1f0ebd1c

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4680&range=04
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4680&range=03-04

  Stats: 22 lines in 9 files changed: 0 ins; 13 del; 9 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4680.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4680/head:pull/4680

PR: https://git.openjdk.java.net/jdk/pull/4680

Reply via email to