On Tue, 3 Dec 2024 10:44:37 GMT, Volkan Yazıcı <d...@openjdk.org> 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

Reply via email to