hg: jdk8/tl/langtools: 8016613: javac should avoid source 8 only analysis when compiling for source 7

2013-06-20 Thread vicente . romero
Changeset: 6debfa63a4a1 Author:vromero Date: 2013-06-20 08:45 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/6debfa63a4a1 8016613: javac should avoid source 8 only analysis when compiling for source 7 Reviewed-by: jjg Contributed-by: maurizio.cimadam...@oracle.com !

Re: 7025238 : HttpURLConnection does not handle URLs with an empty path component

2013-06-20 Thread Chris Hegarty
I see Kurchi pushed this change for you. http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2b156531b7eb For extra credits ;-) does it make sense to something similar on the server side, sun.net.httpserver??? -Chris. On 06/19/2013 07:29 PM, Andreas Rieber wrote: Hi Kurchi, to change the path in

hg: jdk8/tl/jdk: 8016308: Updates to j.u.stream.Node/Nodes

2013-06-20 Thread paul . sandoz
Changeset: 656ea2349aa5 Author:psandoz Date: 2013-06-20 10:45 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/656ea2349aa5 8016308: Updates to j.u.stream.Node/Nodes Reviewed-by: mduigou Contributed-by: Brian Goetz brian.go...@oracle.com, Paul Sandoz paul.san...@oracle.com !

Re: 7025238 : HttpURLConnection does not handle URLs with an empty path component

2013-06-20 Thread Andreas Rieber
Yea, i saw the push and was missing the flowers ;-) For the extra credits i might have some more issues on HttpUrlConnection i could do with your help. And ok, just searched for httpserver on bugs.sun.com - results in 249... Will check what i could do there. cheers Andreas On 20.06.13

hg: jdk8/tl/jdk: 8016324: filter/flatMap pipeline sinks should pass size information to downstream sink

2013-06-20 Thread paul . sandoz
Changeset: 85524d9839dc Author:psandoz Date: 2013-06-20 11:02 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/85524d9839dc 8016324: filter/flatMap pipeline sinks should pass size information to downstream sink Reviewed-by: chegar, mduigou Contributed-by: Brian Goetz

hg: jdk8/tl/jdk: 8016455: Sync stream tests from lambda to tl

2013-06-20 Thread paul . sandoz
Changeset: f758d7c24396 Author:psandoz Date: 2013-06-20 11:15 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f758d7c24396 8016455: Sync stream tests from lambda to tl Reviewed-by: mduigou Contributed-by: Brian Goetz brian.go...@oracle.com, Paul Sandoz paul.san...@oracle.com

Re: 7025238 : HttpURLConnection does not handle URLs with an empty path component

2013-06-20 Thread Andreas Rieber
I see, short test without leading / in the path causes the httpserver to throws exception: Exception in thread main java.lang.IllegalArgumentException: Illegal value for path or protocol at sun.net.httpserver.HttpContextImpl.init(HttpContextImpl.java:60) at

Re: RFR JDK8015799

2013-06-20 Thread Chris Hegarty
Thanks John, I just did a quick test with the testcase attached to the bug report ( below), and the server is indeed replying with a Set-Cookie header with no value ( treated as empty string ) sigh! Since all header retrieval passes through filterHeaderField, in one way or another, I'm a

hg: jdk8/tl/jdk: 8016139: PrimitiveIterator.forEachRemaining

2013-06-20 Thread paul . sandoz
Changeset: 562f5cf13a9c Author:psandoz Date: 2013-06-20 11:21 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/562f5cf13a9c 8016139: PrimitiveIterator.forEachRemaining Reviewed-by: alanb ! src/share/classes/java/util/PrimitiveIterator.java

Re: RFR JDK8014377

2013-06-20 Thread Alan Bateman
On 19/06/2013 17:05, John Zavgren wrote: Greetings: Please consider the following change to the NIO DatagramChannel code that conditionally sets the IP_MULTICAST_ALL socket option to off when the Linux host is running a modern (greater than 2.6.30) kernel. This option is used to prevent

Re: 7025238 : HttpURLConnection does not handle URLs with an empty path component

2013-06-20 Thread Chris Hegarty
On 06/20/2013 10:33 AM, Andreas Rieber wrote: I see, short test without leading / in the path causes the httpserver to throws exception: Exception in thread main java.lang.IllegalArgumentException: Illegal value for path or protocol at

Re: RFR JDK8014499

2013-06-20 Thread Alan Bateman
On 19/06/2013 16:59, John Zavgren wrote: All: Please consider the following change to the MulticastSocket code that conditionally sets the IP_MULTICAST_ALL socket option to off when the Linux host is running a modern (greater than 2.6.30) kernel. This option is used to prevent promiscuous

Re: 7025238 : HttpURLConnection does not handle URLs with an empty path component

2013-06-20 Thread Andreas Rieber
On 20.06.13 12:55, Chris Hegarty wrote: On 06/20/2013 10:33 AM, Andreas Rieber wrote: I see, short test without leading / in the path causes the httpserver to throws exception: Exception in thread main java.lang.IllegalArgumentException: Illegal value for path or protocol at

Re: RFR JDK8015799

2013-06-20 Thread John Zavgren
Chris: Your idea of moving the empty string check so that it's header specific is prudent... that's less likely to have unforeseen consequences. I'll make the change ASAP. John On 06/20/2013 05:56 AM, Chris Hegarty wrote: Thanks John, I just did a quick test with the testcase attached to

Re: RFR JDK8015799

2013-06-20 Thread Chris Hegarty
On 06/20/2013 01:57 PM, John Zavgren wrote: Chris: Your idea of moving the empty string check so that it's header specific is prudent... that's less likely to have unforeseen consequences. I'll make the change ASAP. and maybe a test? -Chris. John On 06/20/2013 05:56 AM, Chris Hegarty

hg: jdk8/tl/jdk: 8017157: catch more exception in test RejectClientRenego

2013-06-20 Thread xuelei . fan
Changeset: a44bd993ce93 Author:xuelei Date: 2013-06-20 07:48 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/a44bd993ce93 8017157: catch more exception in test RejectClientRenego Reviewed-by: vinnie !

Re: RFR JDK8014377

2013-06-20 Thread Chris Hegarty
The changes look fine to me. I can sponsor this for you John. -Chris. On 06/20/2013 11:51 AM, Alan Bateman wrote: On 19/06/2013 17:05, John Zavgren wrote: Greetings: Please consider the following change to the NIO DatagramChannel code that conditionally sets the IP_MULTICAST_ALL socket

Re: RFR JDK8014499

2013-06-20 Thread Chris Hegarty
I do a little clean up, and push the changes for John. -Chris. On 06/20/2013 11:57 AM, Alan Bateman wrote: On 19/06/2013 16:59, John Zavgren wrote: All: Please consider the following change to the MulticastSocket code that conditionally sets the IP_MULTICAST_ALL socket option to off when the

hg: jdk8/tl/jdk: 2 new changesets

2013-06-20 Thread mike . duigou
Changeset: 49b78ec058fb Author:mduigou Date: 2013-06-20 07:23 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/49b78ec058fb 8017088: Map/HashMap.compute() incorrect with key mapping to null value Reviewed-by: dl, dholmes, plevart ! src/share/classes/java/util/HashMap.java !

hg: jdk8/tl/jdk: 8014499: MulticastSocket should enable IP_MULTICAST_ALL (lnx)

2013-06-20 Thread chris . hegarty
Changeset: bf2bacf934d1 Author:chegar Date: 2013-06-20 18:53 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/bf2bacf934d1 8014499: MulticastSocket should enable IP_MULTICAST_ALL (lnx) Reviewed-by: alanb, chegar Contributed-by: John Zavgren john.zavg...@oracle.com, Chris

7157360: HttpURLConnection: HTTP method DELETE doesn't support output

2013-06-20 Thread Andreas Rieber
Hi, here the next issue i would need a sponsor for. Also a very small one. Bug: http://bugs.sun.com/view_bug.do?bug_id=7157360 Looked straight forward to me for the DELETE method, but from last issue i still have the HTTP/1.1 spec open. Here the relevant parts: 4.3 Message Body ... The

hg: jdk8/tl/jdk: 8014377: (dc) DatagramChannel should set IP_MULTICAST_ALL=0 (lnx)

2013-06-20 Thread alan . bateman
Changeset: cd06fc069152 Author:alanb Date: 2013-06-20 19:14 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/cd06fc069152 8014377: (dc) DatagramChannel should set IP_MULTICAST_ALL=0 (lnx) Reviewed-by: chegar, jzavgren ! src/solaris/native/sun/nio/ch/Net.c +