On Tue, 3 Dec 2024 10:44:37 GMT, Volkan Yazıcı <[email protected]> wrote:
> Removes `SecurityManager` and friends from `jdk.httpserver`. Will share
> `tier1..2` results soon.
>
> **Methodology:** Case-insensitively and recursively searched for the
> occurrences of `("removal"|secu|privi)` in `httpserver`-named folders.
Good cleanup!
src/jdk.httpserver/share/classes/sun/net/httpserver/ServerConfig.java line 53:
> 51:
> 52: private static long idleTimerScheduleMillis;
> 53: private static long idleIntervalMillis;
while you're at it you could make these two fields final as well and use
temporary variables in the static block to make sure they're assigned only once.
-------------
PR Review: https://git.openjdk.org/jdk/pull/22510#pullrequestreview-2476336083
PR Review Comment: https://git.openjdk.org/jdk/pull/22510#discussion_r1868117269
