hg: jdk8/tl/jdk: 8025734: Use literal IP address where possible in SocketPermission generated by HttpURLPermission

2013-10-23 Thread michael . x . mcmahon
Changeset: 1b0dfa631b6f Author:michaelm Date: 2013-10-23 11:00 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1b0dfa631b6f 8025734: Use literal IP address where possible in SocketPermission generated by HttpURLPermission Reviewed-by: chegar ! src/share/classes/sun/net/www/p

hg: jdk8/tl/langtools: 8026508: Invokedynamic instructions don't get line number table entries

2013-10-23 Thread jan . lahoda
Changeset: b05db8c815e8 Author:jlahoda Date: 2013-10-23 07:50 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/b05db8c815e8 8026508: Invokedynamic instructions don't get line number table entries Summary: Setting or correcting positions for many trees produced by LambdaT

RFR 8020758: HttpCookie constructor does not throw IAE when name contains a space

2013-10-23 Thread Chris Hegarty
Mark, Michael, java.net.HttpCookie, rightly or wrongly, supports three different Cookie specifications. Some of these are conflicting, and there have been many many bugs reported against various "special" characters being accepted. This one is hopefully straight forward. The issue is specific

Re: RFR 8020758: HttpCookie constructor does not throw IAE when name contains a space

2013-10-23 Thread Michael McMahon
I agree. The change looks fine to me. Thanks Michael On 23/10/13 12:09, Chris Hegarty wrote: Mark, Michael, java.net.HttpCookie, rightly or wrongly, supports three different Cookie specifications. Some of these are conflicting, and there have been many many bugs reported against various "spe

Re: RFR 8020758: HttpCookie constructor does not throw IAE when name contains a space

2013-10-23 Thread Mark Sheppard
Hi Chris, yes, I concur, also regards Mark On 23/10/2013 12:09, Michael McMahon wrote: I agree. The change looks fine to me. Thanks Michael On 23/10/13 12:09, Chris Hegarty wrote: Mark, Michael, java.net.HttpCookie, rightly or wrongly, supports three different Cookie specifications. Some

hg: jdk8/tl/jdk: 8026789: Update test/java/lang/instrument/Re(transform|define)BigClass.sh test to use NMT for memory leak detection

2013-10-23 Thread staffan . larsen
Changeset: 8c20e9ef8709 Author:sla Date: 2013-10-23 15:55 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/8c20e9ef8709 8026789: Update test/java/lang/instrument/Re(transform|define)BigClass.sh test to use NMT for memory leak detection Reviewed-by: dcubed ! test/ProblemList.t

hg: jdk8/tl/jdk: 8020758: HttpCookie constructor does not throw IAE when name contains a space

2013-10-23 Thread chris . hegarty
Changeset: f120672b91ef Author:chegar Date: 2013-10-23 14:38 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f120672b91ef 8020758: HttpCookie constructor does not throw IAE when name contains a space Reviewed-by: michaelm, msheppar ! src/share/classes/java/net/HttpCookie.java

hg: jdk8/tl/jdk: 5036554: unmarshal error on CORBA alias type in CORBA any

2013-10-23 Thread sean . coffey
Changeset: 2af3f5a61322 Author:coffeys Date: 2013-10-23 16:53 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2af3f5a61322 5036554: unmarshal error on CORBA alias type in CORBA any Reviewed-by: chegar, smarks + test/com/sun/corba/5036554/JavaBug.java + test/com/sun/corba/5036

hg: jdk8/tl/corba: 5036554: unmarshal error on CORBA alias type in CORBA any

2013-10-23 Thread sean . coffey
Changeset: a90e9efa4264 Author:coffeys Date: 2013-10-23 16:45 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/a90e9efa4264 5036554: unmarshal error on CORBA alias type in CORBA any Reviewed-by: chegar, smarks ! src/share/classes/com/sun/corba/se/impl/corba/AnyImpl.java

Re: RFR: JDK-7122887 - JDK ignores Gnome3 proxy settings

2013-10-23 Thread Chris Hegarty
Hi Dan, Firstly, thank you for working on this. This is really useful, and I'm glad to see it going into JDK 8. I grabed the patch and built it. All looks fine, and the updates you added to the manual test are really useful to verify the implementation. All in all I'm happy with this change

Re: RFR: JDK-7122887 - JDK ignores Gnome3 proxy settings

2013-10-23 Thread Dan Xu
You are right, Chris. The java class objects initialization should be in the init method. I will update it now. Thanks for the review! -Dan On 10/23/2013 11:21 AM, Chris Hegarty wrote: Hi Dan, Firstly, thank you for working on this. This is really useful, and I'm glad to see it going into J

Re: RFR: JDK-7122887 - JDK ignores Gnome3 proxy settings

2013-10-23 Thread Dan Xu
Hi, I have moved initJavaClass function into Java_sun_net_spi_DefaultProxySelector_init to only initialize the java objects once. The updated webrev can be reviewed at http://cr.openjdk.java.net/~dxu/7122887/webrev.01/ . Thanks! -Dan On

hg: jdk8/tl/jdk: 8026405: javax/xml/ws/clientjar/TestWsImport.java failing on JDK 8 nightly aurora test runs

2013-10-23 Thread sean . coffey
Changeset: ee7f1c78bec7 Author:coffeys Date: 2013-10-23 20:51 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/ee7f1c78bec7 8026405: javax/xml/ws/clientjar/TestWsImport.java failing on JDK 8 nightly aurora test runs Reviewed-by: chegar ! test/javax/xml/ws/clientjar/TestWsImpo

Re: RFR: JDK-7122887 - JDK ignores Gnome3 proxy settings

2013-10-23 Thread Chris Hegarty
On 10/23/2013 07:53 PM, Dan Xu wrote: Hi, I have moved initJavaClass function into Java_sun_net_spi_DefaultProxySelector_init to only initialize the java objects once. The updated webrev can be reviewed at http://cr.openjdk.java.net/~dxu/7122887/webrev.01/

hg: jdk8/tl/langtools: 2 new changesets

2013-10-23 Thread jan . lahoda
Changeset: abc3eaccba73 Author:jlahoda Date: 2013-10-23 23:02 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/abc3eaccba73 8027191: Fix for JDK-8026861 refers to an incorrect bug number Summary: Reverting changeset b05db8c815e8, so that it can be applied again with a co

hg: jdk8/tl/jdk: 8024688: further split Map and ConcurrentMap defaults eliminating looping from Map defaults, Map.merge fixes and doc fixes.

2013-10-23 Thread mike . duigou
Changeset: f4129fcfacdc Author:mduigou Date: 2013-10-23 14:32 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f4129fcfacdc 8024688: further split Map and ConcurrentMap defaults eliminating looping from Map defaults, Map.merge fixes and doc fixes. Reviewed-by: psandoz, dholmes

hg: jdk8/tl/langtools: 8026936: Initialize LamdbaToMethod lazily and as required

2013-10-23 Thread kumar . x . srinivasan
Changeset: 31fe30e2deac Author:ksrini Date: 2013-10-23 15:45 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/31fe30e2deac 8026936: Initialize LamdbaToMethod lazily and as required Reviewed-by: jjg, rfield Contributed-by: jan.lah...@oracle.com ! src/share/classes/com/sun

ProblemList updating request for JDK-7079145: java/net/ipv6tests/UdpTest.java

2013-10-23 Thread Eric Wang
Hi, This is a review request to remove the test java/net/ipv6tests/UdpTest.java from ProblemList.txt. The test tracked by JDK-7079145 is hung if linux kernel is 2.6.18. By upgrading the kernel to higher version 2.6.32, the problem doesn't exi

ProblemList updating request for JDK-7079145: java/net/ipv6tests/UdpTest.java

2013-10-23 Thread Eric Wang
Hi, This is a review request to remove the test java/net/ipv6tests/UdpTest.java from ProblemList.txt. The test tracked by JDK-7079145 is hung if linux kernel is 2.6.18. By upgrading the kernel to higher version 2.6.32, the problem doesn't exi

Re: ProblemList updating request for JDK-7079145: java/net/ipv6tests/UdpTest.java

2013-10-23 Thread Eric Wang
On 2013/10/24 10:57, Eric Wang wrote: Hi, This is a review request to remove the test java/net/ipv6tests/UdpTest.java from ProblemList.txt. The test tracked by JDK-7079145 is hung if linux kernel is 2.6.18. By upgrading the kernel to higher v

hg: jdk8/tl/jdk: 7122887: JDK ignores Gnome3 proxy settings

2013-10-23 Thread dan . xu
Changeset: c9562ac9b812 Author:dxu Date: 2013-10-23 22:30 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c9562ac9b812 7122887: JDK ignores Gnome3 proxy settings Summary: Fix GConf and add to use GProxyResolver to handle network proxy resolution Reviewed-by: chegar ! src/sol