On Mon, 10 Oct 2022 14:19:18 GMT, Sean Mullan <mul...@openjdk.org> wrote:
>> src/java.base/share/classes/java/security/ProtectionDomain.java line 76:
>>
>>> 74: static class JavaSecurityAccessImpl implements JavaSecurityAccess {
>>> 75: /* cache a copy for recording purposes */
>>> 76: static Properties initialSecurityProperties;
>>
>> This doesn't look very clean. Could the Security class hold the initial
>> security properties and provide an accessor method that
>> JavaSecurityAccess:getIinitialProperties could use?
>
> Agree, and alternatively, it seems cleaner to add a new SharedSecrets class
> for `java.security.Security` and remove the dependency on PD.
modified code to have Security class hold the initial properties and provided
an accessor method
-------------
PR: https://git.openjdk.org/jdk/pull/10394
