RFR: 8345139: Fix bugs and inconsistencies in the Provider services map [v2]

Anthony Scarpino ascarpino at openjdk.org
Fri Jan 24 17:51:51 UTC 2025


On Thu, 12 Dec 2024 04:50:04 GMT, Martin Balao <mbalao at openjdk.org> wrote:

>> src/java.base/share/classes/java/security/Provider.java line 710:
>> 
>>> 708:          * Enum to inform the result of an operation on the services map.
>>> 709:          */
>>> 710:         enum SvcOpResult {
>> 
>> Why use an enum here when a boolean is sufficient?
>
> Just to add some more semantic value but we don't have a strong opinion, we can replace it with boolean if you want.

Yes, please do that.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22613#discussion_r1924505761


More information about the security-dev mailing list