RFR: 8343958: Remove security manager impl in java.lang.Process and java.lang.Runtime.exec [v4]
Jaikiran Pai
jpai at openjdk.org
Wed Nov 13 11:02:48 UTC 2024
On Wed, 13 Nov 2024 02:17:23 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> Refactor removing the dependencies on SecurityManager, doPrivileged, and AccessController.
>
> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
>
> Revert unnecessary copyright change in ProcessEnvironment.java.
> Fix code style in ProcessImpl
> Drop unneeded suppress warnings.
> Remove obsolete @throws SecurityExceptions
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.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22024#discussion_r1839985669
More information about the core-libs-dev
mailing list