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