RE: [OpenJDK 2D-Dev] RFR(L): 8024854: Basic changes and files to build the class library on AIX

2013-09-19 Thread Iris Clark
Hi, Volker. Just wanted to say that in addition to preparing for JavaOne (next week!), we're also busy fixing bugs in anticipation of JDK 8 ZBB milestone on 24 Oct [1]. Thanks, iris [1]: http://openjdk.java.net/projects/jdk8/milestones -Original Message- From: Phil Race Sent: Wednesd

hg: jdk8/tl/jdk: 8024405: Spliterators.spliterator should support CONCURRENT characteristic

2013-09-19 Thread paul . sandoz
Changeset: 0ef7ddef9de0 Author:psandoz Date: 2013-09-19 20:41 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/0ef7ddef9de0 8024405: Spliterators.spliterator should support CONCURRENT characteristic Reviewed-by: martin ! src/share/classes/java/util/Spliterator.java ! src/share

hg: jdk8/tl/nashorn: 4 new changesets

2013-09-19 Thread sundararajan . athijegannathan
Changeset: f954d3f4d192 Author:sundar Date: 2013-09-19 13:34 +0530 URL: http://hg.openjdk.java.net/jdk8/tl/nashorn/rev/f954d3f4d192 8025048: true as case label results in ClassCastException Reviewed-by: lagergren ! src/jdk/nashorn/internal/codegen/Attr.java + test/script/basic/JDK-

hg: jdk8/tl/langtools: 8025110: TreeCopier does not correctly copy LabeledStatementTree

2013-09-19 Thread jonathan . gibbons
Changeset: 9a75bdb249a2 Author:jjg Date: 2013-09-19 19:18 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/9a75bdb249a2 8025110: TreeCopier does not correctly copy LabeledStatementTree Reviewed-by: jjg Contributed-by: Werner Dietl ! src/share/classes/com/sun/tools/javac

hg: jdk8/tl/jdk: 8025002: "".codePoints().sorted().iterator().hasNext() causes NegativeArraySizeException

2013-09-19 Thread paul . sandoz
Changeset: f36714707c38 Author:psandoz Date: 2013-09-18 10:49 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f36714707c38 8025002: "".codePoints().sorted().iterator().hasNext() causes NegativeArraySizeException Reviewed-by: henryjen, alanb ! src/share/classes/java/lang/Char

hg: hsx/hotspot-rt/jdk: 96 new changesets

2013-09-19 Thread john . coomes
Changeset: 12ac08d79c9b Author:vadim Date: 2013-08-23 14:13 +0400 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/jdk/rev/12ac08d79c9b 8023052: JVM crash in native layout Reviewed-by: bae, prr ! src/share/native/sun/font/layout/SunLayoutEngine.cpp Changeset: b5c5cff52455 Author:

hg: hsx/hotspot-rt/nashorn: 21 new changesets

2013-09-19 Thread john . coomes
Changeset: b5ff11e00050 Author:sundar Date: 2013-09-04 14:29 +0530 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/nashorn/rev/b5ff11e00050 8024120: Setting __proto__ to null removes the __proto__ property Reviewed-by: lagergren, attila ! src/jdk/nashorn/internal/runtime/ScriptObjec

hg: hsx/hotspot-rt: 14 new changesets

2013-09-19 Thread john . coomes
Changeset: 6d7f27953da6 Author:mduigou Date: 2013-09-03 15:23 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/rev/6d7f27953da6 8024200: handle hg wrapper with space after #! Reviewed-by: tbell ! common/bin/hgforest.sh Changeset: 73355c4c1bc8 Author:lana Date: 2013-09

hg: hsx/hotspot-rt/jaxws: Added tag jdk8-b108 for changeset d1ea68556fd7

2013-09-19 Thread john . coomes
Changeset: f64b1e497722 Author:cl Date: 2013-09-19 09:37 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/jaxws/rev/f64b1e497722 Added tag jdk8-b108 for changeset d1ea68556fd7 ! .hgtags

hg: hsx/hotspot-rt/langtools: 39 new changesets

2013-09-19 Thread john . coomes
Changeset: 23f0f3c9c44a Author:jjg Date: 2013-08-29 19:19 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/langtools/rev/23f0f3c9c44a 8023833: Replace direct use of AnnotatedType in javadoc code Reviewed-by: darcy ! src/share/classes/com/sun/tools/javadoc/AnnotatedTypeImpl.java

hg: jdk8/tl/langtools: 8024437: Inferring the exception thrown: sometimes fails to compile

2013-09-19 Thread vicente . romero
Changeset: 2375ce96e80d Author:vromero Date: 2013-09-19 20:57 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/2375ce96e80d 8024437: Inferring the exception thrown: sometimes fails to compile Reviewed-by: jjg ! src/share/classes/com/sun/tools/javac/code/Flags.java ! src/

hg: hsx/hotspot-rt/jaxp: Added tag jdk8-b108 for changeset 8ade3eed63da

2013-09-19 Thread john . coomes
Changeset: 21b10835b88a Author:cl Date: 2013-09-19 09:37 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/jaxp/rev/21b10835b88a Added tag jdk8-b108 for changeset 8ade3eed63da ! .hgtags

hg: hsx/hotspot-rt/corba: 4 new changesets

2013-09-19 Thread john . coomes
Changeset: af8e5bc3a150 Author:coffeys Date: 2013-09-03 22:35 +0100 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/corba/rev/af8e5bc3a150 8017195: Introduce option to setKeepAlive parameter on CORBA sockets Reviewed-by: chegar, msheppar ! src/share/classes/com/sun/corba/se/impl/tr

Re: RFR(S): JDK-8024985 com/sun/jdi/StepTest.java failed since jdk8b107

2013-09-19 Thread Daniel D. Daugherty
> http://cr.openjdk.java.net/~sla/8024985/webrev.01/ Thumbs up. Of course, the proof is in the running of the tests... :-) test/com/sun/jdi/ExceptionEvents.java No comments (7 excludes here). test/com/sun/jdi/FilterNoMatch.java The original code filtered out "com.*" so the new code filt

Re: RFR(S): JDK-8024985 com/sun/jdi/StepTest.java failed since jdk8b107

2013-09-19 Thread Staffan Larsen
On 19 sep 2013, at 17:49, Daniel D. Daugherty wrote: > On 9/19/13 7:53 AM, Staffan Larsen wrote: >> All, >> >> Please review this fix for a JDI test. The problem and fix is described in >> the bug report. >> >> bug: https://bugs.openjdk.java.net/browse/JDK-8024985 >> webrev: http://cr.openjd

hg: hsx/hotspot-rt/hotspot: 8024517: runtime/CDSCompressedKPtrs/XShareAuto.java failed with RuntimeException

2013-09-19 Thread harold . seigel
Changeset: 9ed97b511b26 Author:hseigel Date: 2013-09-19 11:04 -0400 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/9ed97b511b26 8024517: runtime/CDSCompressedKPtrs/XShareAuto.java failed with RuntimeException Summary: Make sure CDS is off by default when running server c

hg: jdk8/tl/jdk: 8023113: tools/jar/ChangeDir.java fails if /tmp/a exists

2013-09-19 Thread xueming . shen
Changeset: 278873b2b3f8 Author:sherman Date: 2013-09-19 10:06 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/278873b2b3f8 8023113: tools/jar/ChangeDir.java fails if /tmp/a exists Summary: updated the test case Reviewed-by: alanb ! test/tools/jar/ChangeDir.java

Re: RFR(S): JDK-8024985 com/sun/jdi/StepTest.java failed since jdk8b107

2013-09-19 Thread serguei.spit...@oracle.com
Looks good. Thanks, Serguei On 9/19/13 6:53 AM, Staffan Larsen wrote: All, Please review this fix for a JDI test. The problem and fix is described in the bug report. bug: https://bugs.openjdk.java.net/browse/JDK-8024985 webrev: http://cr.openjdk.java.net/~sla/8024985/webrev.00/ Thanks, /Sta

Re: RFR(S): JDK-8024985 com/sun/jdi/StepTest.java failed since jdk8b107

2013-09-19 Thread Daniel D. Daugherty
On 9/19/13 7:53 AM, Staffan Larsen wrote: All, Please review this fix for a JDI test. The problem and fix is described in the bug report. bug: https://bugs.openjdk.java.net/browse/JDK-8024985 webrev: http://cr.openjdk.java.net/~sla/8024985/webrev.00/ Thumbs up. test/com/sun/jdi/TestScaffold

Re: RFR 8022887: Assertion hit while using class and redefining it with RedefineClasses s,imultaneously

2013-09-19 Thread Coleen Phillimore
Thank you Serguei - I made all of the suggested changes below. On 09/19/2013 04:27 AM, serguei.spit...@oracle.com wrote: Hi Coleen, The fix is good, thank you for fixing this! Just a few minor comments below. I'll try to avoid commenting the same spots that Dan has already covered. src/sha

Re: RFR 8022887: Assertion hit while using class and redefining it with RedefineClasses s,imultaneously

2013-09-19 Thread Coleen Phillimore
On 09/18/2013 06:21 PM, Daniel D. Daugherty wrote: On 9/18/13 4:13 PM, Coleen Phillimore wrote: Thank you Dan for the really quick review! No problem. I figured you wanted a quick review with JavaOne approaching and all... On 9/18/2013 5:26 PM, Daniel D. Daugherty wrote: On 9/18/13 1:37

hg: jdk8/tl/jdk: 8011940: java.lang.Class.getAnnotations() always enters synchronized method

2013-09-19 Thread peter . levart
Changeset: 7557062d2dd2 Author:plevart Date: 2013-09-19 16:14 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/7557062d2dd2 8011940: java.lang.Class.getAnnotations() always enters synchronized method Reviewed-by: jfranck, chegar, psandoz, shade ! src/share/classes/java/lang/Cl

hg: jdk8/tl/langtools: 8024609: sjavac assertion fails during call to BuildState.collectArtifacts

2013-09-19 Thread jonathan . gibbons
Changeset: 0cfd5baa7154 Author:ohrstrom Date: 2013-09-19 08:26 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/0cfd5baa7154 8024609: sjavac assertion fails during call to BuildState.collectArtifacts Reviewed-by: jjg ! src/share/classes/com/sun/tools/sjavac/BuildState.ja

Re: jmx-dev RFR: 8004926 sun/management/jmxremote/bootstrap/CustomLauncherTest.sh oftenly times out

2013-09-19 Thread Jaroslav Bachorik
The updated webrev: http://cr.openjdk.java.net/~jbachorik/8004926/webrev.03 I've moved some of the functionality to the testlibrary. -JB - On 12.9.2013 17:31, Jaroslav Bachorik wrote: On 09/12/2013 05:13 PM, Dmitry Samersoff wrote: Jaroslav, CustomLauncherTest.java: 102: this check could be

Re: [OpenJDK 2D-Dev] RFR(L): 8024854: Basic changes and files to build the class library on AIX

2013-09-19 Thread Volker Simonis
Hi Phil, I'm open to any good ideas. The current solution is pragmatic, simple and it has no impact on existing platforms. The other possibility I see is to put the 'dladdr' implantation into its own library (something equivalent to "libjli") and link that library to "libawt". But I currently do

Re: RFR(S): JDK-8024985 com/sun/jdi/StepTest.java failed since jdk8b107

2013-09-19 Thread Jaroslav Bachorik
Looks good. (not-a-reviewer) -JB- On 19.9.2013 15:53, Staffan Larsen wrote: All, Please review this fix for a JDI test. The problem and fix is described in the bug report. bug: https://bugs.openjdk.java.net/browse/JDK-8024985 webrev: http://cr.openjdk.java.net/~sla/8024985/webrev.00/ Thanks

RFR(S): JDK-8024985 com/sun/jdi/StepTest.java failed since jdk8b107

2013-09-19 Thread Staffan Larsen
All, Please review this fix for a JDI test. The problem and fix is described in the bug report. bug: https://bugs.openjdk.java.net/browse/JDK-8024985 webrev: http://cr.openjdk.java.net/~sla/8024985/webrev.00/ Thanks, /Staffan

Re: [OpenJDK 2D-Dev] RFR(L): 8024854: Basic changes and files to build the class library on AIX

2013-09-19 Thread Phil Race
Hi, w.r.t the one issue below : is the awt load library code the only place you need/are doing this ? If someone else (hotspot, core-libs) already assented to this then I guess I could too but I'd like to hear for sure that hotspot and core-libs teams both agree to this approach and whether th

hg: jdk8/tl/jdk: 8017248: Compiler Diacritics Issue

2013-09-19 Thread alexander . zuev
Changeset: 22e9f0067b5a Author:kizune Date: 2013-09-19 17:04 +0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/22e9f0067b5a 8017248: Compiler Diacritics Issue Reviewed-by: naoto ! src/share/classes/sun/launcher/LauncherHelper.java + test/tools/launcher/8017248/ClassÁ.java + t

hg: jdk8/tl/langtools: 8017248: Compiler Diacritics Issue

2013-09-19 Thread alexander . zuev
Changeset: 36e342dd57e2 Author:kizune Date: 2013-09-19 17:05 +0400 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/36e342dd57e2 8017248: Compiler Diacritics Issue Reviewed-by: naoto ! src/share/classes/com/sun/tools/javac/file/RegularFileObject.java

Re: [OpenJDK 2D-Dev] RFR(L): 8024854: Basic changes and files to build the class library on AIX

2013-09-19 Thread Volker Simonis
Hi Phil, thank you for looking at the changes. Please find my answers inline: On Thu, Sep 19, 2013 at 12:22 AM, Phil Race wrote: > Volker, > > I've skimmed the client related changes - mostly based on the descriptions. > I'll look in more detail as soon as I can after JavaOne. We also have som

Re: RFR 8022887: Assertion hit while using class and redefining it with RedefineClasses s,imultaneously

2013-09-19 Thread serguei.spit...@oracle.com
Hi Coleen, The fix is good, thank you for fixing this! Just a few minor comments below. I'll try to avoid commenting the same spots that Dan has already covered. src/share/vm/oops/instanceKlass.hpp 1171 // A pointer to the current info object so we can handle the deletes. 1172 PreviousVe