On Wed, 13 Nov 2024 02:17:23 GMT, Roger Riggs <rri...@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
