Review request for 6817246

Alan Bateman Alan.Bateman at Sun.COM
Wed Mar 18 02:48:18 PDT 2009


Mandy Chung wrote:
> 6817246: Redundant call to set InetAddressCachePolicy to FOREVER if 
> not set during initialization
>
> Webrev at:
>    http://cr.openjdk.java.net/~mchung/6817246/webrev.00/
>
> System.setSecurityManager0 calls 
> InetAddressCachePolicy.setIfNotSet(InetAddressCachePolicy.FOREVER) to 
> set the cache policy to FOREVER if not set by the command-line 
> property.  In fact, this call is redundant since the default cache 
> policy is already set to FOREVER by the static initializer of the 
> InetAddressCachePolicy class.
>
> This fix is to remove the InetAddressCachePolicy.setIfNotSet method 
> call from System.setSecurityManager0.
>
> I ran the io and net jtreg tests.
>
> Thanks
> Mandy
This looks okay to me and I don't see any side effects to have the 
caching policy initialized a bit later.

-Alan.



More information about the net-dev mailing list