Re: HttpCookie.domainMatches("hostname.local", "hostname") return false

2011-03-24 Thread Sean Chou
Hi Chris, Is there any progress on this issue? -- Best Regards, Sean Chou

hg: jdk7/tl/jdk: 7023056: NPE from sun.security.util.ManifestEntryVerifier.verify during Maven build

2011-03-24 Thread weijun . wang
Changeset: 4a64eefbfd7a Author:weijun Date: 2011-03-25 11:58 +0800 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/4a64eefbfd7a 7023056: NPE from sun.security.util.ManifestEntryVerifier.verify during Maven build Reviewed-by: mullan ! src/share/classes/java/util/jar/JarVerifier.jav

hg: jdk7/tl/jdk: 7029680: fix test/sun/misc/Version/Version.java build parsing

2011-03-24 Thread stuart . marks
Changeset: f326a018fd3f Author:smarks Date: 2011-03-24 17:26 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/f326a018fd3f 7029680: fix test/sun/misc/Version/Version.java build parsing Reviewed-by: ohair ! test/sun/misc/Version/Version.java

hg: jdk7/tl/langtools: 6597678: JavaCompiler.getStandardFileManager always uses default charset not the one that user specifies

2011-03-24 Thread jonathan . gibbons
Changeset: 83260b3305ac Author:jjg Date: 2011-03-24 16:14 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/83260b3305ac 6597678: JavaCompiler.getStandardFileManager always uses default charset not the one that user specifies Reviewed-by: mcimadamore ! src/share/classes/

hg: jdk7/tl/jdk: 7029823: (ann) test/java/lang/annotation/package-info.java no longer compiles

2011-03-24 Thread jim . holmlund
Changeset: 632a96f5752d Author:jjh Date: 2011-03-24 11:40 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/632a96f5752d 7029823: (ann) test/java/lang/annotation/package-info.java no longer compiles Summary: Use @Deprecated instead of @java.lang.annotation.Documented Reviewed-by

Re: Code Review 7030256: Cleanup/Remove code supporting old Windows versions (98, NT, 2000, etc)

2011-03-24 Thread Alan Bateman
Chris Hegarty wrote: : You are right, before these changes iphlpapi.dll is not loaded unless NetworkInterface or ResolverConfigurationImpl are called. After these changes iphlpapi.dll is loaded once net.dll is loaded. I think we should use specify the loading of these dll's to be delayed. I

Re: Code Review 7030256: Cleanup/Remove code supporting old Windows versions (98, NT, 2000, etc)

2011-03-24 Thread Chris Hegarty
On 03/24/11 11:49 AM, Alan Bateman wrote: Chris Hegarty wrote: Yes, we will now have a dependency on iphlpapi.dll, but this dll is part of Windows since Windows 2000 and the functions we are using are all available on XP. Does this address your concern, or do you have another issue? The conc

Re: Code Review 7030256: Cleanup/Remove code supporting old Windows versions (98, NT, 2000, etc)

2011-03-24 Thread Alan Bateman
Chris Hegarty wrote: Yes, we will now have a dependency on iphlpapi.dll, but this dll is part of Windows since Windows 2000 and the functions we are using are all available on XP. Does this address your concern, or do you have another issue? The concern is that we load yet another DLL at star

Re: Code Review 7030256: Cleanup/Remove code supporting old Windows versions (98, NT, 2000, etc)

2011-03-24 Thread Chris Hegarty
On 03/24/11 10:55 AM, Alan Bateman wrote: Chris Hegarty wrote: Alan, Michael, The subject line says it all! Nice to be able to remove this old code that is no longer necessary. Hopefully this will make maintenance a little easier going forward. http://cr.openjdk.java.net/~chegar/7030256/webrev

Re: Code Review 7030256: Cleanup/Remove code supporting old Windows versions (98, NT, 2000, etc)

2011-03-24 Thread Alan Bateman
Chris Hegarty wrote: Alan, Michael, The subject line says it all! Nice to be able to remove this old code that is no longer necessary. Hopefully this will make maintenance a little easier going forward. http://cr.openjdk.java.net/~chegar/7030256/webrev.00/webrev/ Mike Duigou noticed I pushe

Code Review 7030256: Cleanup/Remove code supporting old Windows versions (98, NT, 2000, etc)

2011-03-24 Thread Chris Hegarty
Alan, Michael, The subject line says it all! Nice to be able to remove this old code that is no longer necessary. Hopefully this will make maintenance a little easier going forward. http://cr.openjdk.java.net/~chegar/7030256/webrev.00/webrev/ Mike Duigou noticed I pushed this webrev to cr.op