On Sun, 23 Mar 2025 08:47:51 GMT, Chen Liang <li...@openjdk.org> wrote:

>> Volkan Yazici has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Improve `applyVectorMask` JavaDoc
>>   
>>   Co-authored-by: Michael McMahon 
>> <70538289+michael-mc-ma...@users.noreply.github.com>
>
> test/jdk/java/net/httpclient/websocket/Abort.java line 26:
> 
>> 24: /*
>> 25:  * @test
>> 26:  * @modules java.net.http/jdk.internal.net.http.websocket
> 
> Instead of adding this line to all tests, you can add a new `TEST.properties` 
> that contains:
> 
> modules = java.net.http/jdk.internal.net.http.websocket
> 
> And only add `@modules` for files that require extra modules. See other 
> `TEST.properties` in the JDK.

Good call – this indeed reduces the size of touched files. Implemented in 
f5d347157bb8dd8e56457be489fc8899df286f5c.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24033#discussion_r2009811292

Reply via email to