On Wed, 2 Nov 2022 08:19:41 GMT, Andrey Turbanov <[email protected]> wrote:

> Couple of local variables in 
> `com.apple.laf.AquaFileSystemModel.FilesLoader#run` are used only within the 
> method from single thread. So we can avoid usage of legacy synchronized 
> `Vector` here and use `ArrayList` instead.

src/java.desktop/macosx/classes/com/apple/laf/AquaFileSystemModel.java line 428:

> 426:                 }
> 427:                 final DoChangeContents runnable = new 
> DoChangeContents(chunk, fid);
> 428:                 runnables.addElement(runnable);

I think the “chunk” doesn’t need to be a vector either.

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

PR: https://git.openjdk.org/jdk/pull/10942

Reply via email to