RFR: 8345153: Clean up SecurityManager references from jdk.compiler module

Jaikiran Pai jpai at openjdk.org
Thu Nov 28 10:22:10 UTC 2024


Can I please get a review of this change which removes references to SecurityManager related APIs from the `jdk.compiler` module?

With the removal of SecurityManager implementation, various parts of the JDK, including classloader construction or `ServiceLoader` usage will no longer do the `SecurityManager` backed security checks. Failure of such security checks would previously raise a `SecurityException`. With these checks now no longer applicable, the `SecurityException` is treated as any other regular `RuntimeException`. The commit in this PR removes the special treatment of `SecurityException` and other usages of SecurityManager implementation.

No new tests have been added and existing tests in tier1, tier2 and tier3 continue to pass with this change.

-------------

Commit messages:
 - 8345153: Clean up SecurityManager references from jdk.compiler module

Changes: https://git.openjdk.org/jdk/pull/22431/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22431&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8345153
  Stats: 84 lines in 5 files changed: 0 ins; 44 del; 40 mod
  Patch: https://git.openjdk.org/jdk/pull/22431.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22431/head:pull/22431

PR: https://git.openjdk.org/jdk/pull/22431


More information about the compiler-dev mailing list