Re: Request for review & approval: 8141260: isReachable crash in windows xp

2015-11-25 Thread Rob McKenna
Sounds good Mark, will do pre-push, cheers. -Rob On 25/11/15 18:27, Mark Sheppard wrote: yes, the fix looks fine, and I verified the individual isReachable test on the test host where it failed. it is possible to refactor the function Java_java_net_Inet4AddressImpl_isReachable0 a lit

Re: Request for review & approval: 8141260: isReachable crash in windows xp

2015-11-25 Thread Mark Sheppard
yes, the fix looks fine, and I verified the individual isReachable test on the test host where it failed. it is possible to refactor the function Java_java_net_Inet4AddressImpl_isReachable0 a little, by extracting the newly re-introduced "else" block into its own function e.g. wxp_ping4 su

Re: Request for review & approval: 8141260: isReachable crash in windows xp

2015-11-25 Thread Rob McKenna
forgot to cc net-dev -Rob On 24/11/15 16:37, Rob McKenna wrote: Hi folks, The recently updated ICMP (8133015) code fails on Windows XP due to a missing api. This fix allows XP to fall back to the old tcp based method: https://bugs.openjdk.java.net/browse/JDK-8141260 http://cr.openjdk.

Re: Request for review & approval: 8141260: isReachable crash in windows xp

2015-11-25 Thread Seán Coffey
Looks ok to me Rob and provides a re-introduction of the old Java_java_net_Inet4AddressImpl_isReachable0 function for XP systems where necessary. Reviewed. Approved for jdk8u-dev also. Regards, Sean. On 25/11/15 14:00, Rob McKenna wrote: forgot to cc net-dev -Rob On 24/11/15 16:37, Rob