On Thu, 21 Nov 2024 05:24:26 GMT, Jaikiran Pai <j...@openjdk.org> wrote:

> in addition, should we explicitly set this system property to false (or 
> remove the system property) when --patch-module isn't used? I think that 
> would then prevent applications from passing -Djdk.patched=true when the 
> runtime hasn't been patched.

Sure, I can add this.

> While at it, should there be additional guards to prevent applications from 
> changing the value of this system property by doing 
> System.setProperty("jdk.patched") at runtime?

The JVM sets this property as read-only. So setting `jdk.patched=true` at 
command line or in code won't work to override. I'll add tests for these 
scenarios.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22277#discussion_r1851788164

Reply via email to