hg: jdk7/tl/jdk: 6947677: InetAddress.isReachable() throws "java.net.SocketException:Invalid argument" on Linux if run as root

2010-10-22 Thread chris . hegarty
Changeset: 549257d35662 Author:chegar Date: 2010-10-22 09:20 +0100 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/549257d35662 6947677: InetAddress.isReachable() throws "java.net.SocketException:Invalid argument" on Linux if run as root Reviewed-by: alanb ! src/solaris/native/jav

hg: jdk7/tl/jdk: 6816049: (bf) MappedByteBuffer.force() method does not flush data correctly

2010-10-22 Thread alan . bateman
Changeset: 3740c2da7cc5 Author:alanb Date: 2010-10-22 17:40 +0100 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/3740c2da7cc5 6816049: (bf) MappedByteBuffer.force() method does not flush data correctly Reviewed-by: chegar ! src/share/classes/java/nio/Direct-X-Buffer.java.template

Code Review 6994079: PlainSocketImpl should close the socket if it fails

2010-10-22 Thread Chris Hegarty
Alan, Michael, connect(SocketAddress,int) should close the underlying file descriptor if it fails for any reason. This is consistent with the other connect methods. A little other cleanup while there. http://cr.openjdk.java.net/~chegar/6994079/webrev.00/webrev/ -Chris.

Re: Code Review 6994079: PlainSocketImpl should close the socket if it fails

2010-10-22 Thread Alan Bateman
Chris Hegarty wrote: Alan, Michael, connect(SocketAddress,int) should close the underlying file descriptor if it fails for any reason. This is consistent with the other connect methods. A little other cleanup while there. http://cr.openjdk.java.net/~chegar/6994079/webrev.00/webrev/ -Chris.

hg: jdk7/tl/jdk: 6985460: PlatformLogger throws ArrayStoreException when j.u.logging is initialized

2010-10-22 Thread mandy . chung
Changeset: 0fd9c87a9b7b Author:mchung Date: 2010-10-22 11:22 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/0fd9c87a9b7b 6985460: PlatformLogger throws ArrayStoreException when j.u.logging is initialized Reviewed-by: dholmes ! src/share/classes/java/util/logging/LogRecord.j

Re: Code Review 6994079: PlainSocketImpl should close the socket if it fails

2010-10-22 Thread Chris Hegarty
Alan Bateman wrote: Chris Hegarty wrote: Alan, Michael, connect(SocketAddress,int) should close the underlying file descriptor if it fails for any reason. This is consistent with the other connect methods. A little other cleanup while there. http://cr.openjdk.java.net/~chegar/6994079/webrev

hg: jdk7/tl/jdk: 6994079: PlainSocketImpl should close the socket if it fails

2010-10-22 Thread chris . hegarty
Changeset: 0b07344d5526 Author:chegar Date: 2010-10-22 20:27 +0100 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/0b07344d5526 6994079: PlainSocketImpl should close the socket if it fails Reviewed-by: alanb ! src/share/classes/java/net/AbstractPlainSocketImpl.java

hg: jdk7/tl/langtools: 6993301: catch parameters do not have correct kind (i.e. ElementKind.EXCEPTION_PARAMETER)

2010-10-22 Thread jonathan . gibbons
Changeset: 01eabcd240e9 Author:jjg Date: 2010-10-22 14:04 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/01eabcd240e9 6993301: catch parameters do not have correct kind (i.e. ElementKind.EXCEPTION_PARAMETER) Reviewed-by: mcimadamore ! src/share/classes/com/sun/tools/j