Re: RFR: 8297750: Unnecessary Vector usage in IIORegistry

2022-11-29 Thread Sergey Bylokhov
On Sun, 27 Nov 2022 17:51:15 GMT, Andrey Turbanov wrote: > Field `javax.imageio.spi.IIORegistry#initialCategories` is modified only in > `static {}` block, which makes it effectively final. Instead of legacy > synchronized `Vector` we can use non-threadsafe `ArrayList` here. Marked as

Re: RFR: 8297750: Unnecessary Vector usage in IIORegistry

2022-11-29 Thread Phil Race
On Sun, 27 Nov 2022 17:51:15 GMT, Andrey Turbanov wrote: > Field `javax.imageio.spi.IIORegistry#initialCategories` is modified only in > `static {}` block, which makes it effectively final. Instead of legacy > synchronized `Vector` we can use non-threadsafe `ArrayList` here. Marked as

Re: RFR: 8297750: Unnecessary Vector usage in IIORegistry

2022-11-28 Thread Andrey Turbanov
On Mon, 28 Nov 2022 20:01:31 GMT, Sergey Bylokhov wrote: >> src/java.desktop/share/classes/javax/imageio/spi/IIORegistry.java line 89: >> >>> 87: * categories (superinterfaces) to be used in the constructor. >>> 88: */ >>> 89: private static final ArrayList> initialCategories =

Re: RFR: 8297750: Unnecessary Vector usage in IIORegistry

2022-11-28 Thread Sergey Bylokhov
On Sun, 27 Nov 2022 22:22:54 GMT, David Schlosnagle wrote: >> Field `javax.imageio.spi.IIORegistry#initialCategories` is modified only in >> `static {}` block, which makes it effectively final. Instead of legacy >> synchronized `Vector` we can use non-threadsafe `ArrayList` here. > >

Re: RFR: 8297750: Unnecessary Vector usage in IIORegistry

2022-11-28 Thread David Schlosnagle
On Sun, 27 Nov 2022 17:51:15 GMT, Andrey Turbanov wrote: > Field `javax.imageio.spi.IIORegistry#initialCategories` is modified only in > `static {}` block, which makes it effectively final. Instead of legacy > synchronized `Vector` we can use non-threadsafe `ArrayList` here.

RFR: 8297750: Unnecessary Vector usage in IIORegistry

2022-11-28 Thread Andrey Turbanov
Field `javax.imageio.spi.IIORegistry#initialCategories` is modified only in `static {}` block, which makes it effectively final. Instead of legacy synchronized `Vector` we can use non-threadsafe `ArrayList` here. - Commit messages: - 8297750: Unnecessary Vector usage in