hg: jdk8/tl/jdk: 8006637: Failure to filter out native frame events on Solaris

2013-03-19 Thread staffan . larsen
Changeset: a0275a47fa78 Author:sla Date: 2013-03-19 09:53 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/a0275a47fa78 8006637: Failure to filter out native frame events on Solaris Summary: Test is confused by other threads calling String.intern(). Add a thread filter to avoi

hg: jdk8/tl/jdk: 8009426: "profiles" target fails due to nashorn if "images" is not built first

2013-03-19 Thread david . holmes
Changeset: e766da5575fa Author:dholmes Date: 2013-03-19 06:01 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e766da5575fa 8009426: "profiles" target fails due to nashorn if "images" is not built first Reviewed-by: alanb ! makefiles/CreateJars.gmk ! makefiles/Profiles.gmk ! m

RFR 8010282: sun.net.www.protocol.jar.JarFileFactory.close(JarFile) should be thread-safe

2013-03-19 Thread Chris Hegarty
JarFileFactory has two Maps that it uses to implement caching of jar files. Access to these maps should always be done while holding the instance lock, as multiple threads can be simultaneously updating the maps. The close() method was added some time after the original implementation, and it

Re: RFR 8010282: sun.net.www.protocol.jar.JarFileFactory.close(JarFile) should be thread-safe

2013-03-19 Thread Kurchi Hazra
The if statements in the platform specific close methods do not have the spaces as we usually prefer to have them (I realize from before your changes), but otherwise this looks good to me. - Kurchi On 3/19/2013 10:43 AM, Chris Hegarty wrote: JarFileFactory has two Maps that it uses to implemen

Re: RFR 8010282: sun.net.www.protocol.jar.JarFileFactory.close(JarFile) should be thread-safe

2013-03-19 Thread Chris Hegarty
On 03/19/2013 06:16 PM, Kurchi Hazra wrote: The if statements in the platform specific close methods do not have the spaces as we usually prefer to have them (I realize from before your changes), but otherwise this looks good to me. Thanks Kurchi. I will put the space in the correct place befor

Re: RFR 8010282: sun.net.www.protocol.jar.JarFileFactory.close(JarFile) should be thread-safe

2013-03-19 Thread Alan Bateman
On 19/03/2013 17:43, Chris Hegarty wrote: JarFileFactory has two Maps that it uses to implement caching of jar files. Access to these maps should always be done while holding the instance lock, as multiple threads can be simultaneously updating the maps. The close() method was added some time

Re: RFR 8010282: sun.net.www.protocol.jar.JarFileFactory.close(JarFile) should be thread-safe

2013-03-19 Thread Chris Hegarty
Thanks Alan, I agree with both comments and will makes the changes before I push. -Chris. On 19 Mar 2013, at 19:57, Alan Bateman wrote: > On 19/03/2013 17:43, Chris Hegarty wrote: >> JarFileFactory has two Maps that it uses to implement caching of jar files. >> Access to these maps should alw

hg: jdk8/tl/langtools: 8010179: Remove transitional target values from javac

2013-03-19 Thread joe . darcy
Changeset: a4913ea9bb62 Author:darcy Date: 2013-03-19 13:10 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/a4913ea9bb62 8010179: Remove transitional target values from javac Reviewed-by: jjg, mcimadamore ! src/share/classes/com/sun/tools/javac/comp/Attr.java ! src/shar

hg: jdk8/tl/langtools: 8010315: doclint errors in javac public API

2013-03-19 Thread jonathan . gibbons
Changeset: 578eb3dd111d Author:jjg Date: 2013-03-19 15:13 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/578eb3dd111d 8010315: doclint errors in javac public API Reviewed-by: darcy ! make/build.xml ! src/share/classes/com/sun/source/util/DocTreeScanner.java ! src/share

hg: jdk8/tl/jdk: 8001642: Add Optional, OptionalDouble, OptionalInt, OptionalLong

2013-03-19 Thread mike . duigou
Changeset: 2241a2d34085 Author:mduigou Date: 2013-03-19 16:05 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2241a2d34085 8001642: Add Optional, OptionalDouble, OptionalInt, OptionalLong Reviewed-by: mduigou, darcy, alanb, jjb Contributed-by: Brian Goetz + src/share/classes

hg: jdk8/tl/hotspot: 40 new changesets

2013-03-19 Thread lana . steuck
Changeset: 8196357e95b5 Author:amurillo Date: 2013-03-08 08:22 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/hotspot/rev/8196357e95b5 8009688: new hotspot build - hs25-b23 Reviewed-by: jcoomes ! make/hotspot_version Changeset: 255c0a4cb4eb Author:sla Date: 2013-03-05 08:5

hg: jdk8/tl/langtools: 8010333: Remove com.sun.tools.javac.Server

2013-03-19 Thread jonathan . gibbons
Changeset: 9cf17b7a5fe7 Author:jjg Date: 2013-03-19 17:05 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/9cf17b7a5fe7 8010333: Remove com.sun.tools.javac.Server Reviewed-by: darcy - src/share/classes/com/sun/tools/javac/Server.java

hg: jdk8/tl/langtools: 8010317: DocLint incorrectly reports some tags as empty

2013-03-19 Thread jonathan . gibbons
Changeset: 74d7f9bcac93 Author:jjg Date: 2013-03-19 19:16 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/74d7f9bcac93 8010317: DocLint incorrectly reports some tags as empty Reviewed-by: darcy ! src/share/classes/com/sun/tools/doclint/Checker.java + test/tools/doclint