RFR: 8323624: ProviderList.ServiceList does not need to be a list [v3]
Weijun Wang
weijun at openjdk.org
Tue Mar 5 14:55:07 UTC 2024
On Tue, 5 Mar 2024 14:24:35 GMT, Sean Mullan <mullan at openjdk.org> 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
More information about the security-dev
mailing list