Integrated: 8344289: SM cleanup in jdk.internal.util

[Eirik Bjørsnøs]

On Fri, 15 Nov 2024 09:35:17 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:

> Please review this PR to clean up `RandomSupport`, `ClassFileDumper` and `StaticProperty` in the `jdk.internal.util` namespace:
> 
> * `RandomSupport` is updated to replace an `AccessController::doPrivileged` call with `Boolean::getBoolean`. (Existing code uses `String::equalsIgnoreCase`, equivalent to `Boolean::getBoolean`)
> * `ClassFileDumper` constructor is updated to remove a comment referencing `GetPropertyAction`. (I left the `VM::getSavedProperty` call as-is, please advise if this should be replaced with `System::getProperty)
> * `ClassFileDumper::write` is updated to unwrap a `AccessController::doPrivileged` call
> * `StaticProperty` is updated to remove `SecurityManager::checkPropertyAccess` references in the documentation
> 
> Verification: GHA results pending.

This pull request has now been integrated.

Changeset: d0b770c9
Author:    Eirik Bjørsnøs <eirbjo at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/d0b770c938be6b5b3a0176148265ef33184a9b8b
Stats:     118 lines in 3 files changed: 6 ins; 91 del; 21 mod

8344289: SM cleanup in jdk.internal.util

Reviewed-by: liach, rriggs, bpb

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

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