Code Review request, JDK-6645409, Remove not used DefaultHostnameVerifier

Xuelei Fan xuelei.fan at oracle.com
Tue Jul 25 17:56:13 UTC 2017


Updated.  Thanks for the quick review.

Xuelei

On 7/25/2017 10:54 AM, Sean Mullan wrote:
> The bug needs a noreg-cleanup label. Also, I would change the title to 
> "Remove unused DefaultHostNameVerifier".
> 
> Looks fine otherwise.
> 
> --Sean
> 
> On 7/25/17 1:47 PM, Xuelei Fan wrote:
>> Hi,
>>
>> Please review a code cleanup update:
>>
>>     https://bugs.openjdk.java.net/browse/JDK-6645409
>>
>> The sun.net.www.protocol.https.DefaultHostnameVerifier is not used any 
>> more. We can remove it safely now.
>>
>> % hg remove 
>> src/java.base/share/classes/sun/net/www/protocol/https/DefaultHostnameVerifier.java 
>>
>>
>> % hg status 
>> src/java.base/share/classes/sun/net/www/protocol/https/DefaultHostnameVerifier.java 
>>
>> R 
>> src/java.base/share/classes/sun/net/www/protocol/https/DefaultHostnameVerifier.java 
>>
>>
>>
>> Thanks,
>> Xuelei


More information about the security-dev mailing list