On Wed, 27 Nov 2024 14:39:21 GMT, Sean Mullan <mul...@openjdk.org> wrote:
>> Now that JEP 486 has been integrated, `java.security.KeyStore` and >> `Identity` API and impl dependencies on `System.getSecurityManager`, >> `AccessController.doPrivileged` and `AccessControlContext` can be removed. >> >> Most of the changes are straightforward, although some of the refactoring in >> `KeyStore` was a bit complex due to having to extract the code from the >> `PrivilegedAction` subclasses. > > Sean Mullan has updated the pull request incrementally with one additional > commit since the last revision: > > Fix infinite loop which was causing a few test failures. Marked as reviewed by hchao (Committer). Changes look good to me. ------------- PR Review: https://git.openjdk.org/jdk/pull/22401#pullrequestreview-2465469774 PR Comment: https://git.openjdk.org/jdk/pull/22401#issuecomment-2504273720
