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