On Tue, 5 Nov 2024 12:49:04 GMT, Andrey Turbanov <aturba...@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.

The changes look good to me. Also, I've launched existing JNDI/LDAP tests, and 
no issues related to the change were observed.

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

Marked as reviewed by aefimov (Reviewer).

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

Reply via email to