RFR: 8332072: Convert package.html files in `java.naming` to package-info.java [v2]
Nizar Benalla
nbenalla at openjdk.org
Fri Jun 28 10:44:06 UTC 2024
On Thu, 27 Jun 2024 10:27:03 GMT, Aleksei Efimov <aefimov at openjdk.org> wrote:
>> Nizar Benalla has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>>
>> - use javadoc standards
>> - Merge remote-tracking branch 'upstream/master' into naming
>> - remove whitespace
>> - 8332072: Convert package.html files in `java.naming` to package-info.java
>
> src/java.naming/share/classes/javax/naming/ldap/spi/package-info.java line 29:
>
>> 27: *
>> 28: * Extends the {@code javax.naming.ldap} package to provide the classes
>> 29: * and interfaces for the Service Provider Interface (SPI) for LDAP
>
> The `javax.naming.ldap.spi` package only contains SPI classes that can customize DNS lookups when performing LDAP operations. It was added by [JDK-8160768](https://bugs.openjdk.org/browse/JDK-8160768)/[CSR](https://bugs.openjdk.org/browse/JDK-8192975).
> Since the package only contains classes related to this SPI maybe we could change the wording to something like this:
> Suggestion:
>
> * Provides the Service Provider Interface for DNS lookups when
> * performing LDAP operations.
Thanks, Fixed in [0104e4a](https://github.com/openjdk/jdk/pull/19529/commits/0104e4ac5a9bfe143822e72756771c29e0f32731)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19529#discussion_r1658495820
More information about the core-libs-dev
mailing list