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

Aleksei Efimov aefimov at openjdk.java.net
Tue Sep 15 10:01:58 UTC 2020


On Tue, 15 Sep 2020 09:39:29 GMT, Daniel Fuchs <dfuchs 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

Hi Christoph,

Looks good to me.

Kind Regards,
Aleksei

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

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


More information about the core-libs-dev mailing list