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

Eirik Bjørsnøs eirbjo at openjdk.org
Fri Nov 15 16:25:48 UTC 2024


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.

@liach Does this mean we can/should use `System.getProperty` here?

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

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


More information about the core-libs-dev mailing list