Where to discuss class proposals?

2011-01-04 Thread Markus KARG
Dear Core Library Developers, >From discussing with other coders for 25+ years, since the early days of Java, and from having contributed to several Java EE subprojects (like JAX-RS and others) in the past, and from the feedback I got to several of my blog postings on java.net, there turned out

6993732: Remove the HPI

2011-01-04 Thread Alan Bateman
The HPI is left over from bygone years where we had a portability layer and needed to support green and native threading models. Some of the code has not been used or maintained since the transition to HotSpot back in jdk1.3. As of jdk7-b123, the HPI library (libhpi.so/hpi.dll) isn't loaded a

Re: 6993732: Remove the HPI

2011-01-04 Thread Lance Andersen - Oracle
looks fine to me Alan On Jan 4, 2011, at 8:50 AM, Alan Bateman wrote: > > The HPI is left over from bygone years where we had a portability layer and > needed to support green and native threading models. Some of the code has not > been used or maintained since the transition to HotSpot back in

Re: 6993732: Remove the HPI

2011-01-04 Thread Chris Hegarty
Looks fine. -Chris. On 04/01/2011 13:50, Alan Bateman wrote: The HPI is left over from bygone years where we had a portability layer and needed to support green and native threading models. Some of the code has not been used or maintained since the transition to HotSpot back in jdk1.3. As of j

Re: Code Review 7000511: PrintStream, PrintWriter, Formatter leave files open when exception thrown

2011-01-04 Thread Chris Hegarty
On 23/12/2010 20:29, Mike Duigou wrote: Feedback is on webrev.02 http://cr.openjdk.java.net/~chegar/7000511/webrev.02/webrev/ : * PrintStream - .flush(), close(), most println() methods synchronize on this for their entire implementation. They could just be made synchronized methods. I thi

Re: Code Review 7000511: PrintStream, PrintWriter, Formatter leave files open when exception thrown

2011-01-04 Thread Chris Hegarty
On 16/12/2010 11:46, Alan Bateman wrote: Chris Hegarty wrote: : Thanks for your comments Alan. I reworked the changes to include them. Updated webrev: http://cr.openjdk.java.net/~chegar/7000511/webrev.01/webrev/ This looks better. A few comments: In PrintStream it looks like the charset wi

hg: jdk7/tl/jdk: 7010192: InetAddress.isReachable hits ShouldNotReachHere with hs20-b04 (win)

2011-01-04 Thread alan . bateman
Changeset: 6f2d19ef67b1 Author:alanb Date: 2011-01-04 17:49 + URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/6f2d19ef67b1 7010192: InetAddress.isReachable hits ShouldNotReachHere with hs20-b04 (win) Reviewed-by: chegar ! src/windows/native/java/net/Inet4AddressImpl.c ! src/win

Re: 6993732: Remove the HPI

2011-01-04 Thread Mandy Chung
On 01/04/11 05:50, Alan Bateman wrote: The HPI is left over from bygone years where we had a portability layer and needed to support green and native threading models. Some of the code has not been used or maintained since the transition to HotSpot back in jdk1.3. As of jdk7-b123, the HPI li

Re: 6993732: Remove the HPI

2011-01-04 Thread mark . reinhold
At last! Your changes look good to me. Thanks for getting this done. - Mark

Re: Code Review 7000511: PrintStream, PrintWriter, Formatter leave files open when exception thrown

2011-01-04 Thread Mike Duigou
On Jan 4 2011, at 07:03 , Chris Hegarty wrote: > On 23/12/2010 20:29, Mike Duigou wrote: >> Feedback is on webrev.02 >> http://cr.openjdk.java.net/~chegar/7000511/webrev.02/webrev/ : >> >> * PrintStream >> >> - .flush(), close(), most println() methods synchronize on this for their >> entire

hg: jdk7/tl/jdk: 7009618: regression test failed caused by the fix for 7003462

2011-01-04 Thread xueming . shen
Changeset: 4379c762ec50 Author:sherman Date: 2011-01-04 14:17 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/4379c762ec50 7009618: regression test failed caused by the fix for 7003462 Summary: avoid caching the "ended" Inflater in ZipFile class Reviewed-by: alanb ! src/share

hg: jdk7/tl/corba: 6962318: Update copyright year

2011-01-04 Thread lana . steuck
Changeset: f90b3e014e83 Author:ohair Date: 2010-12-28 15:52 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/corba/rev/f90b3e014e83 6962318: Update copyright year Reviewed-by: xdono ! make/Makefile ! make/com/sun/corba/minclude/com_sun_corba_se_impl_io.jmk ! make/common/Defs-linux.gmk

hg: jdk7/tl: 6 new changesets

2011-01-04 Thread lana . steuck
Changeset: 4346ba98938b Author:ohair Date: 2010-12-21 16:44 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/rev/4346ba98938b 6360517: ALT_MSDEVTOOLS_PATH and rc.exe location, and rebase location Reviewed-by: ksrini ! Makefile ! README-builds.html Changeset: dc9eb519c6ed Author:o

hg: jdk7/tl/hotspot: 7 new changesets

2011-01-04 Thread lana . steuck
Changeset: cd8189d84e92 Author:trims Date: 2010-12-22 19:20 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/hotspot/rev/cd8189d84e92 Added tag hs20-b04 for changeset 9669f9b28410 ! .hgtags Changeset: b03260081e9b Author:johnc Date: 2010-12-17 11:26 -0800 URL: http://h

hg: jdk7/tl/jaxp: 2 new changesets

2011-01-04 Thread lana . steuck
Changeset: 57ed1f3bec72 Author:ohair Date: 2010-12-28 15:52 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/jaxp/rev/57ed1f3bec72 6962318: Update copyright year Reviewed-by: xdono ! build.properties ! make/Makefile Changeset: 3a829f5ba9d2 Author:lana Date: 2011-01-04 16:28

hg: jdk7/tl/jaxws: 2 new changesets

2011-01-04 Thread lana . steuck
Changeset: 86f60e5b3975 Author:ohair Date: 2010-12-28 15:53 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/jaxws/rev/86f60e5b3975 6962318: Update copyright year Reviewed-by: xdono ! build.properties ! jaxws.properties ! make/Makefile Changeset: 466e89f7e5be Author:lana Date:

hg: jdk7/tl/langtools: 2 new changesets

2011-01-04 Thread lana . steuck
Changeset: 4868a36f6fd8 Author:ohair Date: 2010-12-28 15:54 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/4868a36f6fd8 6962318: Update copyright year Reviewed-by: xdono ! make/Makefile ! make/build.properties ! make/tools/CompileProperties/CompileProperties.java ! mak