RFR: 8344023: Unnecessary Hashtable usage in LdapClient.defaultBinaryAttrs

Tue Nov 12 14:34:15 UTC 2024

On Tue, 5 Nov 2024 12:49:04 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

> Content of Hashtable `com.sun.jndi.ldap.LdapClient#defaultBinaryAttrs` is fully initialized in `<clinit>` block.
> It means we can replace legacy synchronized `Hashtable` with immutable set.

Looks worthwhile and reasonable. What tests did you run?

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

PR Review: https://git.openjdk.org/jdk/pull/21903#pullrequestreview-2429771866