Re: Code Review Request: 7152856: TEST_BUG: sun/net/www/protocol/jar/B4957695.java failing on Windows

2012-04-17 Thread Alan Bateman
On 16/04/2012 22:18, Kurchi Hazra wrote: Hi, Thanks for the reviews. Here is an updated webrev: http://cr.openjdk.java.net/~khazra/7152856/webrev.01 Looks okay to me. I guess is and os could be locals but it's a minor comment (and no need to generate another webrev if you decide to move

Re: Code Review Request: 7152856: TEST_BUG: sun/net/www/protocol/jar/B4957695.java failing on Windows

2012-04-17 Thread Chris Hegarty
On 16/04/12 22:18, Kurchi Hazra wrote: Hi, Thanks for the reviews. Here is an updated webrev: http://cr.openjdk.java.net/~khazra/7152856/webrev.01 The updated webrev looks ok, but the canned HTTP response looks funny. Each HTTP header must be followed by a CRLF ( '\r\n' ), and the end of

Re: Code Review Request: 7015981: java.net.HttpCookie.domainMatches returns false if domain doesn't start with a dot

2012-04-17 Thread Dmeetry Degrave
Thanks Chris. Right, RFC itself contains the statements that can lead to ambiguity and including examples from different sections of RFC w/o a proper context doesn't help with understanding what the method should do and what shouldn't. I'll go ahead and push the fix. thanks, dmeetry On

Re: Code Review Request: 7152856: TEST_BUG: sun/net/www/protocol/jar/B4957695.java failing on Windows

2012-04-17 Thread Chris Hegarty
ship it! -Chris. On 17/04/12 19:27, Kurchi Hazra wrote: Updated webrev: http://cr.openjdk.java.net/~khazra/7152856/webrev.02/ Alan: On second thoughts, both is and os were not required at all. I removed them. - Kurchi On 4/17/2012 8:47 AM, Kurchi Subhra Hazra wrote: I think the HTTP spec

Code Review Request: 7158636: InterfaceAddress.getBroadcast() returns invalid broadcast address on WLAN

2012-04-17 Thread Kurchi Hazra
Hi, In Windows Vista and later, InterfaceAddress.getBroadcast() returns 0.0.0.0 , since these platforms return IF_TYPE_IEEE80211 instead of MIB_IF_TYPE_ETHERNET for wireless interface type now. The fix is to handle IF_TYPE_IEEE80211 in the relevant switch statement. While doing this, I also

Re: Code Review Request: 7158636: InterfaceAddress.getBroadcast() returns invalid broadcast address on WLAN

2012-04-17 Thread Chris Hegarty
On 17/04/12 19:47, Kurchi Hazra wrote: Hi, In Windows Vista and later, InterfaceAddress.getBroadcast() returns 0.0.0.0 , since these platforms return IF_TYPE_IEEE80211 instead of MIB_IF_TYPE_ETHERNET for wireless interface type now. The fix is to handle IF_TYPE_IEEE80211 in the relevant switch

hg: jdk8/tl/jdk: 7157893: Warnings Cleanup in java.util.*

2012-04-17 Thread kurchi . subhra . hazra
Changeset: 1757f049e8c0 Author:khazra Date: 2012-04-17 12:21 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1757f049e8c0 7157893: Warnings Cleanup in java.util.* Summary: Minor code changes to cleanup warnings in java.util.* Reviewed-by: mduigou, naoto, smarks

Re: Code Review Request: 7158636: InterfaceAddress.getBroadcast() returns invalid broadcast address on WLAN

2012-04-17 Thread Chris Hegarty
On 17/04/12 20:14, Alan Bateman wrote: On 17/04/2012 19:47, Kurchi Hazra wrote: Hi, In Windows Vista and later, InterfaceAddress.getBroadcast() returns 0.0.0.0 , since these platforms return IF_TYPE_IEEE80211 instead of MIB_IF_TYPE_ETHERNET for wireless interface type now. The fix is to handle

Re: Code Review Request: 7158636: InterfaceAddress.getBroadcast() returns invalid broadcast address on WLAN

2012-04-17 Thread Kurchi Hazra
Updated webrev: http://cr.openjdk.java.net/~khazra/7158636/webrev.01/ Windows has a more secure version of snprintf and I used that: http://msdn.microsoft.com/en-us/library/f30dzcf6%28v=vs.110%29.aspx Thanks, Kurchi On 4/17/2012 12:24 PM, Chris Hegarty wrote: On 17/04/12 20:14, Alan