On Wed, 26 Jun 2024 18:08:21 GMT, Nizar Benalla <nbena...@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/package-info.java line 200:
>
>> 198: * if (respCtls != null) {
>> 199: * // Find the one we want
>> 200: * for (int i = 0; i < respCtls.length; i++) {
>
> This is the only change that isn't a direct 1:1 conversion, I hope this ok.
This seems reasonable for what is an obvious but minor bug
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19529#discussion_r1655501908
