Re: [PATCH] Review Request - Test bug: 6948101 java/rmi/transport/pinLastArguments/PinLastArguments.java failing intermittently

2012-07-03 Thread Stuart Marks
This approach looks good to me. Please go ahead and update 7123972 as well, and I'll post these webrevs tomorrow (my time). Thanks. s'marks On 7/2/12 11:41 PM, Eric Wang wrote: David, I have added the comments before the loop, please help to review. If it is OK, I'll update the 7123972 as we

Re: Patch review request - Test bug 7123972 test/java/lang/annotation/loaderLeak/Main.java fails intermittently

2012-07-03 Thread Eric Wang
Hi Stuart, I have made updates which is same as 6948101, please help to review, Thanks a lot! Regards, Eric On 2012/6/30 6:01, Stuart Marks wrote: I've posted the updated webrev here: http://cr.openjdk.java.net/~smarks/yiming.wang/7123972/webrev.1/ This code is pretty much the same as 69481

Re: classlist.* and jar_reorder ?

2012-07-03 Thread Kelly Ohair
not sure anyone ever created docs on it it lists the hot classes as i recall and the intention was to force a particular class order in rt.jar for performance reasons, like the mapfiles forced hot native functions into fewer pages on solaris There is a JarReorder class or build tool in the make

hg: jdk8/tl/jdk: 7133344: Document the java.security.properties system property feature in the java.security file

2012-07-03 Thread sean . mullan
Changeset: d375ea39ce9c Author:mullan Date: 2012-07-03 14:56 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d375ea39ce9c 7133344: Document the java.security.properties system property feature in the java.security file Reviewed-by: hawtin, mullan, weijun Contributed-by: jason

Re: [PATCH] Review Request - Test bug: 6948101 java/rmi/transport/pinLastArguments/PinLastArguments.java failing intermittently

2012-07-03 Thread Stuart Marks
Webrev now posted at http://cr.openjdk.java.net/~smarks/yiming.wang/6948101/webrev.2/ The changes look good to me. If there's no further discussion, I'll push them in a couple days (U.S. holiday coming up). s'marks On 7/3/12 12:20 AM, Stuart Marks wrote: This approach looks good to me. Plea

Re: Patch review request - Test bug 7123972 test/java/lang/annotation/loaderLeak/Main.java fails intermittently

2012-07-03 Thread Stuart Marks
Webrev posted here: http://cr.openjdk.java.net/~smarks/yiming.wang/7123972/webrev.2/ The changes look good. If there's no further discussion, I'll push them in a couple days. s'marks On 7/3/12 3:37 AM, Eric Wang wrote: Hi Stuart, I have made updates which is same as 6948101, please help to

Re: [PATCH] Review Request - Test bug: 6948101 java/rmi/transport/pinLastArguments/PinLastArguments.java failing intermittently

2012-07-03 Thread David Holmes
Please fix the missing space in if(ref.get() before pushing. Thanks, David On 4/07/2012 7:04 AM, Stuart Marks wrote: Webrev now posted at http://cr.openjdk.java.net/~smarks/yiming.wang/6948101/webrev.2/ The changes look good to me. If there's no further discussion, I'll push them in a coupl

Re: [PATCH] Review Request - Test bug: 6948101 java/rmi/transport/pinLastArguments/PinLastArguments.java failing intermittently

2012-07-03 Thread Stuart Marks
Sure, I can take care of that. If this is the only change, no need to generate another webrev, Eric. s'marks On 7/3/12 4:53 PM, David Holmes wrote: Please fix the missing space in if(ref.get() before pushing. Thanks, David On 4/07/2012 7:04 AM, Stuart Marks wrote: Webrev now posted at h

Concurrent formatters

2012-07-03 Thread Diego Belfer
Hello, I was thinking of working on trying to convert the code of some of the formatters DecimalFormatter, SimpleDateFormatter, etc to allow concurrent calls to the format and parse methods. I know it has been discussed many times in the forums and I understand they were not initially designed for

Re: [PATCH] Review Request - Test bug: 6948101 java/rmi/transport/pinLastArguments/PinLastArguments.java failing intermittently

2012-07-03 Thread Eric Wang
Opps, It is my carelessness, I will be more careful in the next bug fixes. Thank you to review and change it. Regards, Eric On 2012/7/4 8:58, Stuart Marks wrote: Sure, I can take care of that. If this is the only change, no need to generate another webrev, Eric. s'marks On 7/3/12 4:53 PM, D

hg: jdk8/tl: Added tag jdk8-b45 for changeset 633f2378c904

2012-07-03 Thread lana . steuck
Changeset: 27fa766a2298 Author:katleman Date: 2012-06-28 09:32 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/rev/27fa766a2298 Added tag jdk8-b45 for changeset 633f2378c904 ! .hgtags

hg: jdk8/tl/langtools: Added tag jdk8-b45 for changeset e111e4587cca

2012-07-03 Thread lana . steuck
Changeset: 4ca599497172 Author:katleman Date: 2012-06-28 09:33 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/4ca599497172 Added tag jdk8-b45 for changeset e111e4587cca ! .hgtags

hg: jdk8/tl/jdk: 2 new changesets

2012-07-03 Thread lana . steuck
Changeset: 8d2ed9d58453 Author:katleman Date: 2012-06-28 09:33 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/8d2ed9d58453 Added tag jdk8-b45 for changeset b92353a01aa0 ! .hgtags Changeset: 8ccbd5aabeab Author:lana Date: 2012-07-03 18:24 -0700 URL: http://hg.

hg: jdk8/tl/jaxp: 2 new changesets

2012-07-03 Thread lana . steuck
Changeset: 300f45e99064 Author:katleman Date: 2012-06-28 09:32 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jaxp/rev/300f45e99064 Added tag jdk8-b45 for changeset 57476f66e13c ! .hgtags Changeset: 9cb8be5e6119 Author:lana Date: 2012-07-03 18:24 -0700 URL: http://hg

hg: jdk8/tl/jaxws: Added tag jdk8-b45 for changeset e80ac58b5ba9

2012-07-03 Thread lana . steuck
Changeset: ae368a83c240 Author:katleman Date: 2012-06-28 09:32 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jaxws/rev/ae368a83c240 Added tag jdk8-b45 for changeset e80ac58b5ba9 ! .hgtags

hg: jdk8/tl/hotspot: 33 new changesets

2012-07-03 Thread lana . steuck
Changeset: a8b9798c1d45 Author:katleman Date: 2012-06-28 09:32 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/hotspot/rev/a8b9798c1d45 Added tag jdk8-b45 for changeset 9d5f20961bc5 ! .hgtags Changeset: 1c280e5b8d31 Author:amurillo Date: 2012-06-15 14:17 -0700 URL: ht

hg: jdk8/tl/corba: 2 new changesets

2012-07-03 Thread lana . steuck
Changeset: 30141e598d72 Author:katleman Date: 2012-06-28 09:32 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/30141e598d72 Added tag jdk8-b45 for changeset 747dad9e9d37 ! .hgtags Changeset: 666efea2df67 Author:lana Date: 2012-07-03 18:23 -0700 URL: http://h

hg: jdk8/tl/jdk: 7180038: regression test failure, SSLEngineBadBufferArrayAccess.java

2012-07-03 Thread xuelei . fan
Changeset: 3ae91286f313 Author:xuelei Date: 2012-07-03 20:29 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3ae91286f313 7180038: regression test failure, SSLEngineBadBufferArrayAccess.java Reviewed-by: weijun ! test/sun/security/ssl/com/sun/net/ssl/internal/ssl/SSLEngineIm