Code Review Request: 6328537: Improve javadocs for Socket class by adding references to SocketOptions

2013-05-10 Thread Kurchi Hazra
Hi, This is a simple improvement in the javadocs of the Socket and ServerSocket classes, to cross-reference the SocketOptions class where appropriate. It looks like a lot of changes in the webrev, only because I have re-justified the text in some cases. Bug: http://bugs.sun.com/view_bug.d

hg: jdk8/tl/langtools: 8014365: Restore Objects.requireNonNull(T, Supplier)

2013-05-10 Thread joe . darcy
Changeset: 1c43236f6d69 Author:darcy Date: 2013-05-10 14:31 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/1c43236f6d69 8014365: Restore Objects.requireNonNull(T, Supplier) Reviewed-by: jjg ! makefiles/BuildLangtools.gmk

hg: jdk8/tl/jdk: 8014357: Minor refactorings to sun.reflect.generics.reflectiveObjects.*

2013-05-10 Thread joe . darcy
Changeset: f84b5498b2bb Author:darcy Date: 2013-05-10 12:25 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f84b5498b2bb 8014357: Minor refactorings to sun.reflect.generics.reflectiveObjects.* Reviewed-by: mchung ! src/share/classes/sun/reflect/generics/reflectiveObjects/Gen

hg: jdk8/tl/jdk: 2 new changesets

2013-05-10 Thread mike . duigou
Changeset: 2490769abdfa Author:mduigou Date: 2013-05-10 09:51 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2490769abdfa 8014316: Use Method Refs in j.u.stream.MatchOps Reviewed-by: dholmes ! src/share/classes/java/util/stream/MatchOps.java Changeset: 9891e4d7d5b3 Author:

Re: Non-standard socket options?

2013-05-10 Thread Brian Call
Thanks for your help guys! It would be most excellent to have that socket option turned off by default. It would certainly make multicast receivers written in java behave the same way across all the major platforms. Not sure why this wasn't considered a bug in Linux and, in fact, it's vehementl

hg: jdk8/tl/langtools: 8014318: tools/javac/profiles/ProfileOptionTest.java needs modifying now that javax.script is in compact1

2013-05-10 Thread alan . bateman
Changeset: ce7e1674eb73 Author:alanb Date: 2013-05-10 16:10 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/ce7e1674eb73 8014318: tools/javac/profiles/ProfileOptionTest.java needs modifying now that javax.script is in compact1 Reviewed-by: mchung ! test/tools/javac/pro

hg: jdk8/tl/jdk: 8011128: (fs) Files.createDirectory fails if the resolved path is exactly 248 characters long

2013-05-10 Thread alan . bateman
Changeset: 09a3b08c986f Author:alanb Date: 2013-05-10 14:53 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/09a3b08c986f 8011128: (fs) Files.createDirectory fails if the resolved path is exactly 248 characters long Reviewed-by: khazra, chegar ! src/windows/classes/sun/nio/fs

Re: Code Review Request: 8014254: Selector in HttpServer introduces a 1000 ms delay when using KeepAlive

2013-05-10 Thread Chris Hegarty
Thanks Kurchi, the changes look fine to me. -Chris. On 09/05/2013 22:15, Kurchi Hazra wrote: I have re-arranged the code a little, so that the events are handled first, any freed up connections are then registered with the selector, post which select() is called. This removes the bottleneck in

Code review: 8010464: Evolve java networking same origin policy

2013-05-10 Thread Michael McMahon
Hi, This is the webrev for the HttpURLPermission addition. As well as the new permission class, the change includes the use of the permission in java.net.HttpURLConnection. The code basically checks for a HttpURLPermission in plainConnect(), getInputStream() and getOutputStream() for the request

Re: Non-standard socket options?

2013-05-10 Thread Alan Bateman
On 09/05/2013 21:49, Brian Call wrote: Hi Guys, My name is Brian Call and I'm a software developer for Sotera Wireless. I'm currently developing a relatively complex multicast application using the jdk7 selector-based I/O for multicast and I've run in to a pretty major hurdle. By "complex", I