RFR (S): 8006400: Add support for defining trace types in closed code

2013-01-16 Thread Erik Helin
Hi all, this change adds supports for defining trace types in closed code. Webrev: http://cr.openjdk.java.net/~ehelin/8006400/webrev.00/ Testing: JPRT Thanks, Erik

hg: jdk8/tl/jdk: 8005926: Merge ThreadLocalRandom state into java.lang.Thread

2013-01-16 Thread chris . hegarty
Changeset: 9d8ef6174cfd Author:dl Date: 2013-01-16 10:14 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/9d8ef6174cfd 8005926: Merge ThreadLocalRandom state into java.lang.Thread Reviewed-by: shade, chegar ! src/share/classes/java/lang/Thread.java ! src/share/classes/java/uti

Re: RFR (S): 8006400: Add support for defining trace types in closed code

2013-01-16 Thread Erik Helin
Hi, Staffan Larsen pointed out an error in the file trace.xml - the types were included after the events. The following updated webrev corrects this issue: http://cr.openjdk.java.net/~ehelin/8006400/webrev.01/ Thanks, Erik On 01/16/2013 10:52 AM, Erik Helin wrote: Hi all, this change adds

Re: RFR (S): 8006400: Add support for defining trace types in closed code

2013-01-16 Thread Nils Loodin
I think it looks good. /Nisse On Jan 16, 2013, at 10:52 , Erik Helin wrote: > Hi all, > > this change adds supports for defining trace types in closed code. > > Webrev: > http://cr.openjdk.java.net/~ehelin/8006400/webrev.00/ > > Testing: > JPRT > > Thanks, > Erik

Re: RFR (S): 8006400: Add support for defining trace types in closed code

2013-01-16 Thread Erik Helin
Hi Nils, thanks for reviewing this code! Please see the updated webrev that I just posted in response to an issue pointed out by Staffan Larsen. Thanks, Erik On 01/16/2013 11:37 AM, Nils Loodin wrote: I think it looks good. /Nisse On Jan 16, 2013, at 10:52 , Erik Helin wrote: Hi all,

Re: RFR (S): 8006400: Add support for defining trace types in closed code

2013-01-16 Thread Nils Loodin
Darn, and here I was silently hoping you wouldn't draw attention to my premature review ;) /Nisse On Jan 16, 2013, at 12:00 , Erik Helin wrote: > Hi Nils, > > thanks for reviewing this code! > > Please see the updated webrev that I just posted in response to an issue > pointed out by Staffa

Re: RFR (S): 8006400: Add support for defining trace types in closed code

2013-01-16 Thread Staffan Larsen
Looks good! /Staffan On 16 jan 2013, at 11:38, Erik Helin wrote: > Hi, > > Staffan Larsen pointed out an error in the file trace.xml - the types were > included after the events. > > The following updated webrev corrects this issue: > http://cr.openjdk.java.net/~ehelin/8006400/webrev.01/ >

hg: jdk8/tl/jdk: 8005389: Backout fix for JDK-6500133

2013-01-16 Thread sean . mullan
Changeset: c7d54f93d3e5 Author:juh Date: 2013-01-16 09:51 -0500 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c7d54f93d3e5 8005389: Backout fix for JDK-6500133 Reviewed-by: mullan ! src/share/classes/sun/security/x509/URIName.java ! test/sun/security/x509/URIName/Parse.java

hg: jdk8/tl/langtools: 2 new changesets

2013-01-16 Thread maurizio . cimadamore
Changeset: f785dcac17b7 Author:mcimadamore Date: 2013-01-16 16:27 + URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/f785dcac17b7 8005854: Add support for array constructor references Summary: Support constructor references of the kind int[]::new Reviewed-by: jjg ! src/sha

hg: jdk8/tl/langtools: 8005964: Regression: difference in error recovery after ambiguity causes JCK test failure

2013-01-16 Thread maurizio . cimadamore
Changeset: 1afdf1f1472b Author:mcimadamore Date: 2013-01-16 17:40 + URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/1afdf1f1472b 8005964: Regression: difference in error recovery after ambiguity causes JCK test failure Summary: Wrong implementation of ResolveError.access

hg: jdk8/tl/langtools: 8006236: doclint: structural issue hidden

2013-01-16 Thread jonathan . gibbons
Changeset: 6b6311a8c9cc Author:jjg Date: 2013-01-16 10:29 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/6b6311a8c9cc 8006236: doclint: structural issue hidden Reviewed-by: darcy ! src/share/classes/com/sun/tools/doclint/Checker.java + test/tools/doclint/EndTagsTest.ja

hg: jdk8/tl/jdk: 8005939: sun/security/x509/{X509CRLImplX509CertImpl}/Verify.java fail in confusing way when some providers not present

2013-01-16 Thread sean . mullan
Changeset: f7f77bdf248b Author:juh Date: 2013-01-16 13:35 -0500 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f7f77bdf248b 8005939: sun/security/x509/{X509CRLImplX509CertImpl}/Verify.java fail in confusing way when some providers not present Reviewed-by: mullan, weijun ! test/su

hg: jdk8/tl/corba: 2 new changesets

2013-01-16 Thread lana . steuck
Changeset: cb40427f4714 Author:katleman Date: 2013-01-03 12:44 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/cb40427f4714 Added tag jdk8-b71 for changeset 8171d23e914d ! .hgtags Changeset: 191afde59e7b Author:katleman Date: 2013-01-10 09:55 -0800 URL: http

hg: jdk8/tl/jaxp: 3 new changesets

2013-01-16 Thread lana . steuck
Changeset: bdf2af722a6b Author:katleman Date: 2013-01-03 12:44 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jaxp/rev/bdf2af722a6b Added tag jdk8-b71 for changeset 499be952a291 ! .hgtags Changeset: 84946404d1e1 Author:katleman Date: 2013-01-10 09:55 -0800 URL: http:

hg: jdk8/tl/langtools: 4 new changesets

2013-01-16 Thread lana . steuck
Changeset: 6f0986ed9b7e Author:katleman Date: 2013-01-03 12:44 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/6f0986ed9b7e Added tag jdk8-b71 for changeset 467e4d9281bc ! .hgtags Changeset: 45fed5cfd1c3 Author:katleman Date: 2013-01-10 09:56 -0800 URL:

hg: jdk8/tl/jaxws: 2 new changesets

2013-01-16 Thread lana . steuck
Changeset: d9707230294d Author:katleman Date: 2013-01-03 12:44 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jaxws/rev/d9707230294d Added tag jdk8-b71 for changeset f577a39c9fb3 ! .hgtags Changeset: c606f644a5d9 Author:katleman Date: 2013-01-10 09:55 -0800 URL: http

hg: jdk8/tl/jdk: 6 new changesets

2013-01-16 Thread lana . steuck
Changeset: 32a57e645e01 Author:katleman Date: 2013-01-03 12:44 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/32a57e645e01 Added tag jdk8-b71 for changeset 2a5af0f766d0 ! .hgtags Changeset: c9a914b11436 Author:katleman Date: 2013-01-10 09:55 -0800 URL: http:/

hg: jdk8/tl/langtools: 8006283: Change to Class.cast() in javax.lang.model implementation for repeating annotations

2013-01-16 Thread joe . darcy
Changeset: 8b749558767b Author:darcy Date: 2013-01-16 13:22 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/8b749558767b 8006283: Change to Class.cast() in javax.lang.model implementation for repeating annotations Reviewed-by: jjg ! src/share/classes/com/sun/tools/java

Review Request (S) 8005128: JSR 292: the mlvm redefineClassInBootstrap test crashes in ConstantPool::compare_entry_to

2013-01-16 Thread [email protected]
Please, review the fix for: https://jbs.oracle.com/bugs/browse/JDK-8005128 Open webrev: http://cr.openjdk.java.net/~sspitsyn/webrevs/2013/hotspot/8005128-JVMTI-JSR292.0/ Summary: When constant pool is copied by the merge_constant_pools() the invokedynamic operands must be copied in a right