On Mon, 27 Apr 2026 08:23:59 GMT, Mikhail Yankelevich 
<[email protected]> wrote:

>> Artur Barashev has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Typo fix
>
> test/jdk/javax/net/ssl/ServerName/SNIWildcardMatching.java line 60:
> 
>> 58: /*
>> 59:  * @test
>> 60:  * @bug 8381380
> 
> Since this is a test only bug, there is no need for a bug id here. Could you 
> please remove it?

Done, thanks!

> test/jdk/javax/net/ssl/ServerName/SNIWildcardMatching.java line 69:
> 
>> 67:  *          java.base/sun.security.util
>> 68:  *
>> 69:  * @run main SNIWildcardMatching
> 
> nit: don't thinks this is doing anything here. If you prefer to keep it I'm 
> ok with that too

I know, but I prefer it this way for clarity.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30927#discussion_r3150031273
PR Review Comment: https://git.openjdk.org/jdk/pull/30927#discussion_r3150017879

Reply via email to