On Wed, 19 Nov 2025 13:32:00 GMT, Jaikiran Pai <[email protected]> wrote:
>> src/java.net.http/share/classes/jdk/internal/net/http/Http2Connection.java
>> line 38:
>>
>>> 36: import java.net.http.HttpHeaders;
>>> 37: import java.nio.ByteBuffer;
>>> 38: import java.nio.channels.NetworkChannel;
>>
>> Unless I'm mistaken this import is not needed?
>
> The javadoc comment I added to `Http2Connection.isOpen()` uses `{@linkplain
> NetworkChannel#isOpen() channel is open}`. It looks like the IDE added it for
> that reason. I'm going to see if removing the import will cause any build
> failures in any places.
Ah - OK - I missed that - the import is needed then.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28233#discussion_r2542046983
