RFR: 8288628: Unnecessary Hashtable usage in ConditionalSpecialCasing

Naoto Sato naoto at openjdk.org
Fri Jun 17 15:45:59 UTC 2022


On Wed, 15 Jun 2022 19:49:52 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

> If a thread-safe implementation is not needed, it is recommended to use HashMap in place of Hashtable.
> `ConditionalSpecialCasing.entryTable` is read-only Map which is modified only in `static` block. It means we can safely replace it with HashMap.

Looks good.
I would have refactored the initialization of the static map itself, but it would be for another day.

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

Marked as reviewed by naoto (Reviewer).

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


More information about the core-libs-dev mailing list