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