Re: RFR 8150521: Test test/java/net/InetAddress/getOriginalHostName.java fails to load JavaNetInetAddressAccess from SharedSecrets

2016-03-03 Thread Chris Hegarty
On 3 Mar 2016, at 09:40, Vyom Tewari wrote: > i need sponsor for this fix as well. Pushed: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/eb5798a009cc -Chris. > Vyom > > On 3/3/2016 2:34 PM, Chris Hegarty wrote: >> On 3 Mar 2016, at 05:33, Vyom Tewari wrote: >> >>> incorporated the review co

Re: RFR 8150521: Test test/java/net/InetAddress/getOriginalHostName.java fails to load JavaNetInetAddressAccess from SharedSecrets

2016-03-03 Thread Vyom Tewari
i need sponsor for this fix as well. Vyom On 3/3/2016 2:34 PM, Chris Hegarty wrote: On 3 Mar 2016, at 05:33, Vyom Tewari wrote: incorporated the review comment, updated webrev in place. http://cr.openjdk.java.net/~nkumar/vyom/8150521/webrev0.1/index.html

Re: RFR 8150521: Test test/java/net/InetAddress/getOriginalHostName.java fails to load JavaNetInetAddressAccess from SharedSecrets

2016-03-03 Thread Chris Hegarty
On 3 Mar 2016, at 09:40, Vyom Tewari wrote: > i need sponsor for this fix as well. I will sponsor this for you. -Chris. > Vyom > > On 3/3/2016 2:34 PM, Chris Hegarty wrote: >> On 3 Mar 2016, at 05:33, Vyom Tewari wrote: >> >>> incorporated the review comment, updated webrev in place. >>>

Re: RFR 8150521: Test test/java/net/InetAddress/getOriginalHostName.java fails to load JavaNetInetAddressAccess from SharedSecrets

2016-03-03 Thread Chris Hegarty
On 3 Mar 2016, at 05:33, Vyom Tewari wrote: > incorporated the review comment, updated webrev in place. > > http://cr.openjdk.java.net/~nkumar/vyom/8150521/webrev0.1/index.html > Looks fine. -Chris. > Vyom > > On 3/2/

Re: RFR 8150521: Test test/java/net/InetAddress/getOriginalHostName.java fails to load JavaNetInetAddressAccess from SharedSecrets

2016-03-02 Thread Vyom Tewari
incorporated the review comment, updated webrev in place. http://cr.openjdk.java.net/~nkumar/vyom/8150521/webrev0.1/index.html Vyom On 3/2/2016 1:45 PM, Alan Bateman wrote: On 02/03/2016 07:44, vyom wrote: Hi Chris, T

Re: RFR 8150521: Test test/java/net/InetAddress/getOriginalHostName.java fails to load JavaNetInetAddressAccess from SharedSecrets

2016-03-02 Thread Alan Bateman
On 02/03/2016 07:44, vyom wrote: Hi Chris, Thanks for review please find the updated webrev(http://cr.openjdk.java.net/~nkumar/vyom/8150521/webrev0.1/index.html ). This looks okay to me. Style wide, can you put a space i

Re: RFR 8150521: Test test/java/net/InetAddress/getOriginalHostName.java fails to load JavaNetInetAddressAccess from SharedSecrets

2016-03-02 Thread Chris Hegarty
> On 2 Mar 2016, at 07:44, vyom wrote: > > Hi Chris, > > Thanks for review please find the updated > webrev(http://cr.openjdk.java.net/~nkumar/vyom/8150521/webrev0.1/index.html > ). Looks good to me Vyom. -Chris. > Tha

Re: RFR 8150521: Test test/java/net/InetAddress/getOriginalHostName.java fails to load JavaNetInetAddressAccess from SharedSecrets

2016-03-01 Thread vyom
Hi Chris, Thanks for review please find the updated webrev(http://cr.openjdk.java.net/~nkumar/vyom/8150521/webrev0.1/index.html ). Thanks, Vyom On Tuesday 01 March 2016 03:35 PM, Chris Hegarty wrote: On 1 Mar 2016, at 0

Re: RFR 8150521: Test test/java/net/InetAddress/getOriginalHostName.java fails to load JavaNetInetAddressAccess from SharedSecrets

2016-03-01 Thread Chris Hegarty
On 1 Mar 2016, at 09:18, vyom 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/815

RFR 8150521: Test test/java/net/InetAddress/getOriginalHostName.java fails to load JavaNetInetAddressAccess from SharedSecrets

2016-03-01 Thread vyom
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