On Thu, 12 Dec 2024 04:50:04 GMT, Martin Balao <mba...@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
