On Thu, 16 Oct 2025 20:07:44 GMT, Josiah Noel <[email protected]> wrote:

>> Now ExchangeImpl will default to having a separate attribute map for the 
>> request duration.
>
> Josiah Noel has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Fix assertion statement in ExchangeAttributeTest

src/jdk.httpserver/share/classes/module-info.java line 27:

> 25: 
> 26: import module jdk.httpserver;
> 27: 

The statement above seems to be causing a NullPointerException in the javadoc 
tool. I've filed a bug on it to find out what we need to do. But, it may mean 
reverting to "import com.sun.net.httpserver.*;"

Also, the two implNotes are combined in the output HTML. So I would re-order 
them so the existing one comes first.

In the meantime I will file a CSR with the additional doc text below

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27652#discussion_r2448681838

Reply via email to