RFR: 8273098: Unnecessary Vector usage in java.naming
Aleksei Efimov
aefimov at openjdk.java.net
Mon Aug 30 11:32:36 UTC 2021
On Thu, 26 Aug 2021 07:04:45 GMT, Andrey Turbanov <github.com+741251+turbanoff at openjdk.org> wrote:
> Usage of thread-safe collection Vector is unnecessary. It's recommended to use ArrayList/array 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.
Hi Andrey,
The changes look good to me.
I've run your patch through our CI and all JNDI tests are green.
Best,
Aleksei
-------------
Marked as reviewed by aefimov (Committer).
PR: https://git.openjdk.java.net/jdk/pull/5262
More information about the core-libs-dev
mailing list