On Fri, 15 Nov 2024 16:15:02 GMT, Chen Liang <li...@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
