RFR: 8343958: Remove security manager impl in java.lang.Process and java.lang.Runtime.exec [v4]

Alan Bateman alanb at openjdk.org
Wed Nov 13 11:42:17 UTC 2024


On Wed, 13 Nov 2024 10:58:41 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> src/java.base/windows/classes/java/lang/ProcessImpl.java line 516:
>> 
>>> 514:         }
>>> 515: 
>>> 516:         return null;
>> 
>> I think this is an oversight - returning `null` from the constructor would be an compile error.
>
> Actually, looking at the GitHub actions job, it appears to have caught one other compilation error in this file:
> 
> 
> jdk\jdk\src\java.base\windows\classes\java\lang\ProcessImpl.java:423: error: ';' expected
>         final String value = System.getProperty("jdk.lang.Process.allowAmbiguousCommands", "true"));
>                                                                                                   ^
> 1 error

I assume Roger will run at least tier1 on all platforms before integrating.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22024#discussion_r1840100951


More information about the core-libs-dev mailing list