RFR: 8253155: Minor cleanups and Javadoc fixes for LdapDnsProvider of java.naming

Daniel Fuchs dfuchs at openjdk.java.net
Tue Sep 15 09:42:22 UTC 2020


On Tue, 15 Sep 2020 07:47:54 GMT, Christoph Langer <clanger at openjdk.org> wrote:

> There are some little flaws in LdapDNSProvider and auxilliary classes, mostly in Javadoc.
> 
> In detail:
> src/java.naming/share/classes/com/sun/jndi/ldap/DefaultLdapDnsProvider.java: Unnecessary import
> src/java.naming/share/classes/com/sun/jndi/ldap/LdapDnsProviderService.java: typo
> src/java.naming/share/classes/javax/naming/ldap/spi/LdapDnsProvider.java: Whitespace
> src/java.naming/share/classes/javax/naming/ldap/spi/LdapDnsProviderResult.java: Spelling of "ldap" -> should be
> capitalized

Hi Christoph,

The changes look good to me.

best regards,

-- daniel

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

Marked as reviewed by dfuchs (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/168


More information about the core-libs-dev mailing list