hg: hsx/hotspot-rt/hotspot: 8004823: Add VM support for type annotation reflection

2012-12-20 Thread stefan . karlsson
Changeset: 35431a769282 Author:stefank Date: 2012-12-20 10:22 +0100 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/35431a769282 8004823: Add VM support for type annotation reflection Reviewed-by: dholmes, coleenp Contributed-by: joel.fra...@oracle.com ! make/bsd/makefil

hg: jdk8/tl/jdk: 6783290: MBeanInfo/MBeanFeatureInfo has inconsistent readObject/writeObject

2012-12-20 Thread dmitry . samersoff
Changeset: b600d490dc57 Author:dsamersoff Date: 2012-12-20 16:02 +0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/b600d490dc57 6783290: MBeanInfo/MBeanFeatureInfo has inconsistent readObject/writeObject Summary: call readObject in all cases Reviewed-by: emcmanus Contributed-by:

[PATCH] JDK-7199324: IPv6 JMXConnectorServer.getConnectionIDs() return IDs contradicting address grammar

2012-12-20 Thread Jaroslav Bachorik
Looking for reviewers and a sponsor. Webrev is available at http://cr.openjdk.java.net/~jbachorik/JDK-7199324/webrev.00 The issue is about the JMX client connection ID not being generated accordingly to the specification when including an IPv6 address. The specification states that in such case t

Re: RR(S) Re: JEP 168: Network Discovery of Manageable Java Processes

2012-12-20 Thread Jaroslav Bachorik
Hi, only a nit-pick: On line 106, 107 might be better to use "autodiscovery is true OR (autodiscovery is not set AND jdp.port is set)" to make the wording consistent. Cheers, -JB- On 12/19/2012 02:36 PM, Dmitry Samersoff wrote: > Hi Everybody, > > Please review JEP changes. > > http://cr.open

Re: RFR JDK-8005120

2012-12-20 Thread Chris Hegarty
On 19/12/2012 20:52, David Holmes wrote: Real sense of deja-vu here. Didn't we go through this same thing with the HPI socket routines? Yes, and the networking native code too. I think it is best to use socklen_t for the unix code. From what I can see making these changes, to use socklen_t, s

hg: jdk8/tl/jdk: 6937053: RMI unmarshalling errors in ClientNotifForwarder cause silent failure

2012-12-20 Thread dmitry . samersoff
Changeset: e43f90d5af11 Author:dsamersoff Date: 2012-12-20 16:56 +0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e43f90d5af11 6937053: RMI unmarshalling errors in ClientNotifForwarder cause silent failure Summary: the catch block in the fetchNotifs() method is extended to expe

hg: jdk8/tl/jdk: 7009998: JMX synchronization during connection restart is faulty

2012-12-20 Thread dmitry . samersoff
Changeset: 3f014bc09297 Author:dsamersoff Date: 2012-12-20 17:24 +0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3f014bc09297 7009998: JMX synchronization during connection restart is faulty Summary: add a return statement after the re-connecting has finished and the state is

hg: jdk8/tl/jdk: 8002356: Add ForkJoin common pool and CountedCompleter

2012-12-20 Thread chris . hegarty
Changeset: d01a810798e0 Author:dl Date: 2012-12-20 13:44 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d01a810798e0 8002356: Add ForkJoin common pool and CountedCompleter Reviewed-by: chegar, mduigou ! make/java/java/FILES_java.gmk + src/share/classes/java/util/concurrent/C

Re: RFR JDK-8005120

2012-12-20 Thread Chris Hegarty
On 20/12/2012 13:49, John Zavgren wrote: Greetings: I agree that the "correct" way to fix this problem is to use POSIX data types, e.g., socklen_t. However, when I switch to the doctrinaire data type, the build fails on windows machines: - build monologue - c:\jprt\t\p1\032220

Re: RFR JDK-8005120

2012-12-20 Thread Dmitry Samersoff
John, On windows socklen_t defined in WS2TCPIP.H make sure it's included when necessary. -Dmitry On 2012-12-20 17:49, John Zavgren wrote: > Greetings: > > I agree that the "correct" way to fix this problem is to use POSIX data > types, e.g., socklen_t. However, when I switch to the doctrinaire

hg: jdk8/tl/jdk: 8005306: Redundant cast warning in KeepAliveStream.java

2012-12-20 Thread chris . hegarty
Changeset: 31d2f9995d6c Author:chegar Date: 2012-12-20 15:04 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/31d2f9995d6c 8005306: Redundant cast warning in KeepAliveStream.java Reviewed-by: alanb ! src/share/classes/sun/net/www/http/KeepAliveStream.java

Re: RFR JDK-8005120

2012-12-20 Thread John Zavgren
Greetings: I agree that the "correct" way to fix this problem is to use POSIX data types, e.g., socklen_t. However, when I switch to the doctrinaire data type, the build fails on windows machines: - build monologue - c:\jprt\t\p1\032220.jzavgren\s\jdk\src\share\transport\socket\s

hg: jdk8/tl/jdk: 8005309: Missed tests for 6783290,6937053,7009998

2012-12-20 Thread dmitry . samersoff
Changeset: c1a55ee9618e Author:dsamersoff Date: 2012-12-20 20:12 +0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c1a55ee9618e 8005309: Missed tests for 6783290,6937053,7009998 Summary: Missed tests for 6783290,6937053,7009998 Reviewed-by: sjiang, emcmanus Contributed-by: jaros

hg: jdk8/tl/jdk: 8001048: JSR-160: Allow IIOP transport to be optional

2012-12-20 Thread alan . bateman
Changeset: edb71a37fcb7 Author:alanb Date: 2012-12-20 20:29 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/edb71a37fcb7 8001048: JSR-160: Allow IIOP transport to be optional Reviewed-by: dsamersoff, dfuchs, mchung ! src/share/classes/com/sun/jmx/remote/internal/IIOPHelper.ja

hg: jdk8/tl/jdk: 8005281: (props) loadFromXML/storeToXML with small parser is not thread safe

2012-12-20 Thread alan . bateman
Changeset: eeda18683ddc Author:alanb Date: 2012-12-20 20:40 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/eeda18683ddc 8005281: (props) loadFromXML/storeToXML with small parser is not thread safe Reviewed-by: mchung ! src/share/classes/jdk/internal/util/xml/BasicXmlProperti

Re: RFR JDK-8005120

2012-12-20 Thread John Zavgren
Greetings: I modified my changes so that windows knows the definition of the POSIX data type: socklen_t, and now all the system calls are using the "doctrinaire" data types. Please consider the following update. http://cr.openjdk.java.net/~mullan/webrevs/jzavgren/8005120/webrev.01/ Thanks! Jo

Re: RFR: 7150256: Add back Diagnostic Command JMX API

2012-12-20 Thread Mandy Chung
Hi Frederic, It's good to see the management interface for diagnostic commands is back. On 12/17/12 6:58 AM, Frederic Parain wrote: http://cr.openjdk.java.net/~fparain/7150256/webrev.01/ Please send your makefiles change to build-dev and build-infra for review to ensure necessary change is m

Re: RFR: 7150256: Add back Diagnostic Command JMX API

2012-12-20 Thread Kelly O'Hair
makefile changes look fine. -kto On Dec 20, 2012, at 3:40 PM, Mandy Chung wrote: > Hi Frederic, > > It's good to see the management interface for diagnostic commands is back. > > On 12/17/12 6:58 AM, Frederic Parain wrote: >> >> >> http://cr.openjdk.java.net/~fparain/7150256/webrev.01/ >>

hg: jdk8/tl/jdk: 8005290: remove -showversion from RMI test library subprocess mechanism

2012-12-20 Thread stuart . marks
Changeset: 60adb69bf043 Author:smarks Date: 2012-12-20 20:11 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/60adb69bf043 8005290: remove -showversion from RMI test library subprocess mechanism Reviewed-by: jgish, chegar, dmocek ! test/java/rmi/testlibrary/JavaVM.java ! test/