On Mon, 5 Jul 2021 14:00:07 GMT, Сергей Цыпанов <github.com+10835776+stsypa...@openjdk.org> wrote:
>> 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. > > src/java.desktop/share/classes/javax/sound/sampled/AudioSystem.java line 244: > >> 242: } >> 243: >> 244: return list.toArray(new Line.Info[list.size()]); > > Shouldn't it be `new Line.Info[0]` as in the class below? It looks fine to change. ------------- PR: https://git.openjdk.java.net/jdk/pull/4680