Hi Chris,

Thanks for review please find the updated webrev(http://cr.openjdk.java.net/~nkumar/vyom/8150521/webrev0.1/index.html <http://cr.openjdk.java.net/%7Enkumar/vyom/8150521/webrev0.1/index.html>).

Thanks,
Vyom

On Tuesday 01 March 2016 03:35 PM, Chris Hegarty wrote:
On 1 Mar 2016, at 09:18, vyom <vyom.tew...@oracle.com> wrote:

Hi All,

Please  review my code changes for testcase fix.

Bug: JDK: Test test/java/net/InetAddress/getOriginalHostName.java fails to load 
JavaNetInetAddressAccess from         SharedSecrets

Webrev: http://cr.openjdk.java.net/~nkumar/vyom/8150521/webrev0.0/index.html 
<http://cr.openjdk.java.net/%7Enkumar/vyom/8150521/webrev0.0/index.html>

This code changes ensures that  InetAdress class is loaded before we call 
"SharedSecrets.getJavaNetInetAddressAccess()”.
I wonder if the test failure is exposing an actual product bug?
Should SharedSecrets.getJavaNetInetAddressAccess ensure that the
InetAddress class is initialized before returning? This is a common
pattern in most of the SS.getXXX methods, and I think it appropriate
here, given its usage.

-Chris.

Reply via email to