On Tue, 27 May 2025 13:25:38 GMT, Mikhail Yankelevich 
<myankelev...@openjdk.org> wrote:

>> HttpServer::stop will terminate the server immidiately after all exhcnages 
>> are complete.
>> If the exchanges take longer then the specified delay it will terminate 
>> straight after the delay, the same as the previous behaviour.
>> 
>> Used to wait until the delay is complete at all times, regardless of the 
>> number of active exchanges.
>> 
>> Tests based on @eirbjo work, so adding Eirik as a contributor.
>
> Mikhail Yankelevich has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   consolidated events

src/jdk.httpserver/share/classes/sun/net/httpserver/Event.java line 35:

> 33:         this.exchange = t;
> 34:     }
> 35: 

I don't see the purpose of this change and the value it brings other than 
collation and reducing explicit files for the derived Events ?
Could you elaborate the rationale please?

I do see a code smell in WriteFinished directly access the state of ExchangeImpl

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/25333#discussion_r2109214767

Reply via email to