On Thu, 3 Jun 2021 14:20:37 GMT, Sean Mullan <mul...@openjdk.org> 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