On Wed, 26 Feb 2025 01:56:58 GMT, Valerie Peng <valer...@openjdk.org> wrote:
>> Francisco Ferrari Bihurriet has updated the pull request incrementally with
>> one additional commit since the last revision:
>>
>> Clear ServicesMap fields in the declared order
>>
>> Constructors assign the fields in the same order.
>
> src/java.base/share/classes/java/security/Provider.java line 750:
>
>> 748: private final Map<ServiceKey, Map<UString, String>>
>> serviceAttrProps;
>> 749:
>> 750: ServicesMap() {
>
> nit: add comment for this constructor?
Ok, I'll add a note.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22613#discussion_r2025536940
