On Fri, 12 Jan 2024 18:06:33 GMT, Weijun Wang <wei...@openjdk.org> wrote:
>> Re-implement it as an `Iterator` to make sure it can only be iterated once
>> and make debugger happy.
>>
>> No regression, just a refactoring.
>
> 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`?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17381#discussion_r1512913289
