RFR: 8344289: SM cleanup in jdk.internal.util [v2]

[Roger Riggs]

On Fri, 15 Nov 2024 16:15:02 GMT, Chen Liang <liach at openjdk.org> wrote:

>> src/java.base/share/classes/jdk/internal/util/ClassFileDumper.java line 82:
>> 
>>> 80: 
>>> 81:     private ClassFileDumper(String key, String path) {
>>> 82:         /*
>> 
>> The comment might still be relevant if changed to System.getProperty.
>
> It's fine now: previously user defined security manager was effectively loading user code that uses java.lang.invoke, now users cannot specify security managers, so this problem does not exist any more.

In that case, the code should be calling System.getProperty and not the backup VM saved props.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22141#discussion_r1844128182