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