RFR: JDK-8308398 Move SunEC crypto provider into java.base [v2]

Anthony Scarpino ascarpino at openjdk.org
Thu Jun 29 19:58:56 UTC 2023


On Thu, 29 Jun 2023 18:36:55 GMT, Valerie Peng <valeriep at openjdk.org> wrote:

>> Anthony Scarpino has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   update for review:  changed test, removed commented out code in module, fixed switch statement, added --limit-modules
>
> src/java.base/share/classes/sun/security/jca/ProviderConfig.java line 235:
> 
>> 233:             provider = p;
>> 234:         }
>> 235:         return provider;
> 
> Why changing it to return 'provider', is this change intentional? IIRC, the general pattern is to return the local reference instead of a volatile variable.

Ok

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14457#discussion_r1247106719


More information about the core-libs-dev mailing list