Re: RFR: 8000450: Restrict access to com/sun/corba/se/impl package

2013-06-05 Thread Alan Bateman
On 04/06/2013 23:57, Seán Coffey wrote: The com.sun.corba.se.impl package is a private package. The API is subject to change at any time and no 3rd party code should be using it. As a result, I'd like to add it to the package restricted list. I've run our internal CORBA SQE testsuite against

hg: jdk8/tl/nashorn: 3 new changesets

2013-06-05 Thread lana . steuck
Changeset: 4463e94d9b0d Author:lana Date: 2013-05-29 16:59 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/nashorn/rev/4463e94d9b0d Merge - src/jdk/nashorn/internal/runtime/options/ValueOption.java - src/jdk/nashorn/internal/runtime/regexp/DefaultRegExp.java -

hg: jdk8/tl: 3 new changesets

2013-06-05 Thread lana . steuck
Changeset: 78852ce176db Author:jqzuo Date: 2013-05-28 20:03 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/rev/78852ce176db 8014762: Add JMC configure option mapping to Jprt.gmk Summary: Need to add the mapping between JPRT env var and configure flag for JMC, from ALT_JMC_ZIP_DIR

hg: jdk8/tl/jaxp: 3 new changesets

2013-06-05 Thread lana . steuck
Changeset: 627c265d6e0c Author:lana Date: 2013-05-29 16:57 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jaxp/rev/627c265d6e0c Merge - src/com/sun/org/apache/xerces/internal/xinclude/ObjectFactory.java - src/com/sun/org/apache/xml/internal/serialize/ObjectFactory.java Changeset:

hg: jdk8/tl/langtools: 3 new changesets

2013-06-05 Thread lana . steuck
Changeset: 18943a1b7a47 Author:lana Date: 2013-05-29 16:59 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/18943a1b7a47 Merge - test/tools/javac/HiddenAbstractMethod/Test - test/tools/javac/NonAmbiguousField/Test Changeset: 2c5a568ee36e Author:lana Date:

hg: jdk8/tl/jdk: 23 new changesets

2013-06-05 Thread lana . steuck
Changeset: 93de1ab38793 Author:jchen Date: 2013-05-17 10:04 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/93de1ab38793 8003444: Fix potential NULL pointer dereference Reviewed-by: jgodinez, prr ! src/share/native/sun/java2d/cmm/lcms/cmscgats.c !

Re: RFR: 8000450: Restrict access to com/sun/corba/se/impl package

2013-06-05 Thread Seán Coffey
On 05/06/2013 07:20, Alan Bateman wrote: On 04/06/2013 23:57, Seán Coffey wrote: The com.sun.corba.se.impl package is a private package. The API is subject to change at any time and no 3rd party code should be using it. As a result, I'd like to add it to the package restricted list. I've run

Re: RFR 8009581: Xpathexception does not honor initcause()

2013-06-05 Thread David Holmes
This seems fine to me. David On 5/06/2013 1:19 AM, Aleksej Efimov wrote: Alan, Jason, Thanks for clarification. I have moved the serial data from binary files to test class: http://cr.openjdk.java.net/~dmeetry/8009581/webrev.5/ http://cr.openjdk.java.net/%7Edmeetry/8009581/webrev.5/ -Aleksej

hg: jdk8/tl/nashorn: 16 new changesets

2013-06-05 Thread sundararajan . athijegannathan
Changeset: 66b2fde90c9d Author:jlaskey Date: 2013-05-29 16:23 -0300 URL: http://hg.openjdk.java.net/jdk8/tl/nashorn/rev/66b2fde90c9d 8015636: Add more typed arrays code coverage tests. Reviewed-by: sundar Contributed-by: james.las...@oracle.com + test/script/basic/typedarrays.js

Re: RFR 8009581: Xpathexception does not honor initcause()

2013-06-05 Thread Alan Bateman
On 04/06/2013 16:19, Aleksej Efimov wrote: Alan, Jason, Thanks for clarification. I have moved the serial data from binary files to test class: http://cr.openjdk.java.net/~dmeetry/8009581/webrev.5/ http://cr.openjdk.java.net/%7Edmeetry/8009581/webrev.5/ -Aleksej Thanks, this looks much

hg: jdk8/tl/jdk: 8003895: java/nio/channels/AsynchronousChannelGroup/Unbounded.java failing again [win64]

2013-06-05 Thread alan . bateman
Changeset: 8a9f897a57d6 Author:alanb Date: 2013-06-05 11:12 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/8a9f897a57d6 8003895: java/nio/channels/AsynchronousChannelGroup/Unbounded.java failing again [win64] Reviewed-by: chegar ! test/ProblemList.txt !

hg: jdk8/tl/jdk: 8013649: HashMap spliterator tryAdvance() encounters remaining elements after forEachRemaining()

2013-06-05 Thread paul . sandoz
Changeset: de11b20f8c01 Author:psandoz Date: 2013-05-31 10:53 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/de11b20f8c01 8013649: HashMap spliterator tryAdvance() encounters remaining elements after forEachRemaining() Reviewed-by: chegar !

8015963: Add at since tags to new ConcurrentHashMap methods

2013-06-05 Thread Chris Hegarty
Just some minor cleanup after the integration of CHMv8. Trivially, add @since 1.8 tags to the 34 new methods and 1 new class in ConcurrentHashMap. http://cr.openjdk.java.net/~chegar/8015963/webrev.00/webrev/src/share/classes/java/util/concurrent/ConcurrentHashMap.java.udiff.html Doug, Here

hg: jdk8/tl/langtools: 7186887: Test T6567415.java can fail on a slow machine

2013-06-05 Thread alexander . zuev
Changeset: 79fd9cfa55f2 Author:kizune Date: 2013-06-05 16:58 +0400 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/79fd9cfa55f2 7186887: Test T6567415.java can fail on a slow machine Reviewed-by: jjg, ksrini ! test/tools/javac/6567415/T6567415.java

hg: jdk8/tl/jdk: 8015880: GenerateBreakIteratorData build warning

2013-06-05 Thread alan . bateman
Changeset: ae700bdb68b6 Author:alanb Date: 2013-06-05 13:57 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/ae700bdb68b6 8015880: GenerateBreakIteratorData build warning Reviewed-by: peytoia ! make/tools/src/build/tools/generatebreakiteratordata/CharSet.java

RFR 8015895 Int/LongStream.range/rangeClosed

2013-06-05 Thread Paul Sandoz
Hi, The following was decided by the JSR-335 EG: 1) Int/LongStream.range are restricted to a fixed step of 1. The map operation can be used to adjust the step. 2) Int/LongStream.rangeClosed methods were added for a closed range with a fixed step of 1, in addition to the half open range of 1).

Re: 8015963: Add at since tags to new ConcurrentHashMap methods

2013-06-05 Thread Aleksey Shipilev
Looks good to me. -Aleksey. On 06/05/2013 02:42 PM, Chris Hegarty wrote: Just some minor cleanup after the integration of CHMv8. Trivially, add @since 1.8 tags to the 34 new methods and 1 new class in ConcurrentHashMap.

Re: 8015963: Add at since tags to new ConcurrentHashMap methods

2013-06-05 Thread Chris Hegarty
On 05/06/2013 14:49, Martin Buchholz wrote: jsr166 CVS updated with this patch, except that the jsr166 style is not to introduce a blank line just for @since. Thanks Martin, Trivially, I think you are still missing a few: reduceEntriesToLong, reduceEntriesToInt, and class KeySetView.

hg: jdk8/tl/jdk: 8011719: Properties.loadFromXML fails with a chunked HTTP connection

2013-06-05 Thread chris . hegarty
Changeset: 5edcc8ca4146 Author:chegar Date: 2013-06-05 16:06 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5edcc8ca4146 8011719: Properties.loadFromXML fails with a chunked HTTP connection Reviewed-by: michaelm !

Re: Measuring performance changes from applying 4837946 patch

2013-06-05 Thread Brian Burkhalter
Hi Sergey, Thanks for the comments and suggestions. On Jun 5, 2013, at 2:20 AM, Aleksey Shipilev wrote: Sergey will follow up on the benchmark itself, I'm just being nit-picking below (not that it helps performance, but it does contribute to cleaner benchmark code): a) if you only have the

hg: jdk8/tl/jdk: 8015963: Add at since tags to new ConcurrentHashMap methods

2013-06-05 Thread chris . hegarty
Changeset: c1af6b5a979a Author:chegar Date: 2013-06-05 16:23 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c1af6b5a979a 8015963: Add at since tags to new ConcurrentHashMap methods Reviewed-by: shade, martin ! src/share/classes/java/util/concurrent/ConcurrentHashMap.java

Re: Measuring performance changes from applying 4837946 patch

2013-06-05 Thread Aleksey Shipilev
On 06/05/2013 07:13 PM, Brian Burkhalter wrote: Hi Sergey, Aleksey :) b) the annotations you are using on each method, can flow up to class-level annotation, and will be inherited by all the @GMB methods; I was not aware of this. Yes, and with the daizy-fresh JMH, it is even possible to

hg: jdk8/tl/jdk: 7150256: Add back Diagnostic Command JMX API

2013-06-05 Thread frederic . parain
Changeset: e857b2a3ecee Author:fparain Date: 2013-06-05 08:41 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e857b2a3ecee 7150256: Add back Diagnostic Command JMX API Reviewed-by: mchung, jbachorik ! make/java/management/Exportedfiles.gmk ! make/java/management/FILES_c.gmk

hg: jdk8/tl/nashorn: 6 new changesets

2013-06-05 Thread james . laskey
Changeset: 0feca8a93cb3 Author:attila Date: 2013-06-05 10:44 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/nashorn/rev/0feca8a93cb3 8015955: ObjectNode.elements should be stronger typed Reviewed-by: lagergren, sundar ! src/jdk/nashorn/internal/codegen/CodeGenerator.java !

Re: Measuring performance changes from applying 4837946 patch

2013-06-05 Thread Brian Burkhalter
On Jun 5, 2013, at 8:26 AM, Aleksey Shipilev wrote: Aleksey : Sorry: I had not finished my first coffee yet. b) the annotations you are using on each method, can flow up to class-level annotation, and will be inherited by all the @GMB methods; I was not aware of this. Yes, and with

Re: RFR 8009581: Xpathexception does not honor initcause()

2013-06-05 Thread Aleksej Efimov
On 06/05/2013 02:09 PM, Alan Bateman wrote: On 04/06/2013 16:19, Aleksej Efimov wrote: Alan, Jason, Thanks for clarification. I have moved the serial data from binary files to test class: http://cr.openjdk.java.net/~dmeetry/8009581/webrev.5/

Re: RFR 8009581: Xpathexception does not honor initcause()

2013-06-05 Thread huizhe wang
Aleksej, When you're ready, after fixing the remaining things as Alan commented, please push the change to hg.openjdk.java.net/jdk8/tl-gate/jaxp. For 7u, send a request to jdk7u-dev and indicate that the changes are identical to that for jdk8. Thanks, Joe On 6/5/2013 3:09 AM, Alan Bateman

Sponsor needed: 7032154: Performance tuning of sun.misc.FloatingDecimal/FormattedFloatingDecimal

2013-06-05 Thread Brian Burkhalter
The webrev and patch are here http://cr.openjdk.java.net/~bpb/7032154/ And the pertinent recent review in this thread http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-May/017317.html The changeset comment is: 7032154: Performance tuning of

Re: Sponsor needed: 7032154: Performance tuning of sun.misc.FloatingDecimal/FormattedFloatingDecimal

2013-06-05 Thread Mandy Chung
I can sponsor it. I haven't got the cycle to review this change but Martin and Iris reviewed it and this changeset is good to go. Mandy On 6/5/2013 10:26 AM, Brian Burkhalter wrote: The webrev and patch are here http://cr.openjdk.java.net/~bpb/7032154/ And the pertinent recent review in

Re: Sponsor needed: 7032154: Performance tuning of sun.misc.FloatingDecimal/FormattedFloatingDecimal

2013-06-05 Thread Brian Burkhalter
Will do. On Jun 5, 2013, at 11:54 AM, Martin Buchholz wrote: You could add more contributors to the Contributed-by: line.

Re: RFR 8009581: Xpathexception does not honor initcause()

2013-06-05 Thread Alan Bateman
On 05/06/2013 17:34, Aleksej Efimov wrote: New webrev: http://cr.openjdk.java.net/~dmeetry/8009581/webrev.7/ http://cr.openjdk.java.net/%7Edmeetry/8009581/webrev.7/ Thanks, you've addressed all the issues so I think we are good now. -Alan

Re: RFR 8015895 Int/LongStream.range/rangeClosed

2013-06-05 Thread Mike Duigou
Looks good to me. Mike On Jun 5 2013, at 06:10 , Paul Sandoz wrote: Hi, The following was decided by the JSR-335 EG: 1) Int/LongStream.range are restricted to a fixed step of 1. The map operation can be used to adjust the step. 2) Int/LongStream.rangeClosed methods were added for a