On Fri, 30 Jan 2026 10:56:47 GMT, Daniel Jeliński <[email protected]> wrote:

>> Daniel Fuchs has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Review feedback
>
> test/jdk/java/net/httpclient/CancelRequestTest.java line 342:
> 
>> 340:     private static CountDownLatch addLatchFor(HttpRequest req) {
>> 341:         // release left over latches
>> 342:         releaseLatches();
> 
> `releaseLatches()` calls `latches.clear()`. We will effectively only have one 
> latch in the map at a time. Is that what you intended?

Yes - we don't know what the key is.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29415#discussion_r2745735136

Reply via email to