On Thu, 3 Jun 2021 14:20:37 GMT, Sean Mullan <[email protected]> wrote:
>> Bradford Wetmore has updated the pull request with a new target base due to
>> a merge or a rebase. The pull request now contains eight commits:
>>
>> - Address codereview comments
>> - Merge branch 'master' into JDK-8267485
>> - Merge branch 'master' into JDK-8267485
>> - Merge branch 'master' into JDK-8267485
>> - Replace missing annotation
>> - Merge branch 'master' into JDK-8267485
>> - Updated copyright date.
>> - 8267485: Remove the dependency on SecurityManager in
>> JceSecurityManager.java
>
> src/java.base/share/classes/javax/crypto/JceSecurityManager.java line 50:
>
>> 48: * @since 1.4
>> 49: */
>> 50: @SuppressWarnings("removal")
>
> You should remove this annotation now that the dependency on SecurityManager
> has been removed.
Unfortunately, we are still calling AccessController, thus the annotation needs
to remain.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4150
