Re: Code review request 7183373: URLClassloader.close() does not close JAR files whose resources have been loaded via getResource()

2013-01-21 Thread Chris Hegarty
On 01/21/2013 08:36 PM, Kurchi Subhra Hazra wrote: The change looks consistent with what we had already (findResource() will now silently consume an UnknownServiceException instead of a NullPointerException for MailToURLConnection). Also, I noticed that the test does not fail on Mac OS X even wi

Re: Code review request 7183373: URLClassloader.close() does not close JAR files whose resources have been loaded via getResource()

2013-01-21 Thread Kurchi Subhra Hazra
The change looks consistent with what we had already (findResource() will now silently consume an UnknownServiceException instead of a NullPointerException for MailToURLConnection). Also, I noticed that the test does not fail on Mac OS X even without the fix - it does fail on Windows though. T

hg: jdk8/tl/langtools: 4 new changesets

2013-01-21 Thread maurizio . cimadamore
Changeset: 7873d37f5b37 Author:mcimadamore Date: 2013-01-21 20:13 + URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/7873d37f5b37 8005244: Implement overload resolution as per latest spec EDR Summary: Add support for stuck expressions and provisional applicability Reviewed-

hg: jdk8/tl/jaxp: 6 new changesets

2013-01-21 Thread lana . steuck
Changeset: cf0917c0d771 Author:katleman Date: 2013-01-16 11:59 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jaxp/rev/cf0917c0d771 Added tag jdk8-b73 for changeset 84946404d1e1 ! .hgtags Changeset: 278a2f60c55b Author:erikj Date: 2013-01-04 11:31 +0100 URL: http://h

hg: jdk8/tl/jdk: 18 new changesets

2013-01-21 Thread lana . steuck
Changeset: 965e89e2abd3 Author:katleman Date: 2013-01-16 12:00 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/965e89e2abd3 Added tag jdk8-b73 for changeset 733885f57e14 ! .hgtags Changeset: e91caf05f441 Author:erikj Date: 2012-12-27 20:18 +0100 URL: http://hg

hg: jdk8/tl/corba: Added tag jdk8-b73 for changeset 191afde59e7b

2013-01-21 Thread lana . steuck
Changeset: 2132845cf5f7 Author:katleman Date: 2013-01-16 11:59 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/2132845cf5f7 Added tag jdk8-b73 for changeset 191afde59e7b ! .hgtags

hg: jdk8/tl/jaxws: 5 new changesets

2013-01-21 Thread lana . steuck
Changeset: 68f508979ffe Author:katleman Date: 2013-01-16 11:59 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jaxws/rev/68f508979ffe Added tag jdk8-b73 for changeset c606f644a5d9 ! .hgtags Changeset: 51f3117e2b75 Author:erikj Date: 2013-01-04 11:31 +0100 URL: http://

hg: jdk8/tl/hotspot: 2 new changesets

2013-01-21 Thread lana . steuck
Changeset: 41ccb2e737fb Author:katleman Date: 2013-01-16 11:59 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/hotspot/rev/41ccb2e737fb Added tag jdk8-b73 for changeset 11619f33cd68 ! .hgtags Changeset: 1a3e54283c54 Author:katleman Date: 2013-01-16 20:53 -0800 URL: ht

hg: jdk8/tl: 28 new changesets

2013-01-21 Thread lana . steuck
Changeset: 4090847a5444 Author:katleman Date: 2013-01-16 11:59 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/rev/4090847a5444 Added tag jdk8-b73 for changeset 93b9664f97ee ! .hgtags Changeset: 77f062a41850 Author:erikj Date: 2012-12-27 20:15 +0100 URL: http://hg.ope

hg: jdk8/tl/langtools: 3 new changesets

2013-01-21 Thread lana . steuck
Changeset: 56c97aff46bb Author:katleman Date: 2013-01-16 12:00 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/56c97aff46bb Added tag jdk8-b73 for changeset 8d0baee36c71 ! .hgtags Changeset: b450959b42ff Author:lana Date: 2013-01-20 23:39 -0800 URL: http

hg: jdk8/tl/jdk: 8006642: Fix javadoc warnings due to Integer.MAX_VALUE

2013-01-21 Thread lance . andersen
Changeset: de30e46250c5 Author:lancea Date: 2013-01-21 14:08 -0500 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/de30e46250c5 8006642: Fix javadoc warnings due to Integer.MAX_VALUE Reviewed-by: alanb ! src/share/classes/java/sql/BatchUpdateException.java ! src/share/classes/java/

hg: jdk8/tl/langtools: 8006263: Supplementary test cases needed for doclint

2013-01-21 Thread jonathan . gibbons
Changeset: 4a3cfc970c6f Author:jjg Date: 2013-01-21 10:00 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/4a3cfc970c6f 8006263: Supplementary test cases needed for doclint Reviewed-by: mcimadamore Contributed-by: peter.jen...@oracle.com ! src/share/classes/com/sun/tools

hg: jdk8/tl/jdk: 8005311: Add Scalable Updatable Variables, DoubleAccumulator, DoubleAdder, LongAccumulator, LongAdder

2013-01-21 Thread chris . hegarty
Changeset: f21a3c273fb2 Author:dl Date: 2013-01-21 13:50 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f21a3c273fb2 8005311: Add Scalable Updatable Variables, DoubleAccumulator, DoubleAdder, LongAccumulator, LongAdder Reviewed-by: chegar, darcy, goetz ! make/java/java/FILE

Re: Code review request 7183373: URLClassloader.close() does not close JAR files whose resources have been loaded via getResource()

2013-01-21 Thread Frank Ding
Hi Chris, Thanks for your review. I did jtreg test on net module (java/net and sun/net) and no regression issue was found. Could anybody else review the patch as well? Best regards, Frank On 1/18/2013 10:55 PM, Chris Hegarty wrote: I haven't been able to spend as much time on this as I wo