On Tue, 15 Sep 2020 07:47:54 GMT, Christoph Langer <clan...@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

Reply via email to