On Tue, 5 Mar 2024 14:24:35 GMT, Sean Mullan <[email protected]> wrote:
>> Weijun Wang has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> use consistent names
>
> src/java.base/share/classes/sun/security/jca/ProviderList.java line 429:
>
>> 427: * Not thread safe.
>> 428: */
>> 429: private final class ServiceList implements Iterator<Service> {
>
> Wouldn't it make more sense to change the name to `ServiceIterator`?
You're right. Fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17381#discussion_r1512964316
