Re: RFR [8022584] Memory leak in some NetworkInterface methods

2013-08-12 Thread Ivan Gerasimov
David, Chris, I reverted back NULL-checking. Now the change consists of one line removal and a regression test. Webrev: http://cr.openjdk.java.net/~igerasim/8022584/6/webrev/ Hg export: http://cr.openjdk.java.net/~igerasim/2commit/8022584-jdk8-Memleak-in-NetworkInterface.patch Sincerely yours

Re: RFR [8022584] Memory leak in some NetworkInterface methods

2013-08-12 Thread Chris Hegarty
Thank you Ivan. This looks good to me. -Chris. P.S. I will give others a chance to comment. If no objections, I will push this tomorrow for you. On 12/08/2013 13:33, Ivan Gerasimov wrote: David, Chris, I reverted back NULL-checking. Now the change consists of one line removal and a regressi

Re: RFR [8022584] Memory leak in some NetworkInterface methods

2013-08-12 Thread Ivan Gerasimov
Thank you Chris! On 12.08.2013 16:43, Chris Hegarty wrote: Thank you Ivan. This looks good to me. -Chris. P.S. I will give others a chance to comment. If no objections, I will push this tomorrow for you. On 12/08/2013 13:33, Ivan Gerasimov wrote: David, Chris, I reverted back NULL-checkin

hg: jdk8/tl/jdk: 2 new changesets

2013-08-12 Thread sean . mullan
Changeset: ffacf3e7a130 Author:mullan Date: 2013-08-12 09:03 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/ffacf3e7a130 8016848: javax_security/auth/login tests fail in compact 1 and 2 profiles Summary: Change the default value of the "login.configuration.provider" security

hg: jdk8/tl/langtools: 2 new changesets

2013-08-12 Thread maurizio . cimadamore
Changeset: f7f271bd74a2 Author:mcimadamore Date: 2013-08-12 17:25 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/f7f271bd74a2 6537020: JCK tests: a compile-time error should be given in case of ambiguously imported fields (types, methods) Summary: Hiding check does not

Re: RFR [8022584] Memory leak in some NetworkInterface methods

2013-08-12 Thread Alan Bateman
On 09/08/2013 13:18, David Holmes wrote: I agree. I'm sure when Alan suggested to check the return he didn't expect it to unravel like this :) As we know hotspot will never actually return NULL there is no urgency to add this in. Sorry about this, I wasn't aware of the issue in the JNI spec, w

hg: jdk8/tl/jdk: 8015780: java/lang/reflect/Method/GenericStringTest.java failing

2013-08-12 Thread vicente . romero
Changeset: 70c8f4a4b8d6 Author:vromero Date: 2013-08-12 17:40 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/70c8f4a4b8d6 8015780: java/lang/reflect/Method/GenericStringTest.java failing Reviewed-by: darcy, jfranck ! test/ProblemList.txt ! test/java/lang/reflect/Method/Gener

hg: jdk8/tl/jdk: 8022753: SQLXML javadoc example typo

2013-08-12 Thread lance . andersen
Changeset: cc64a05836a7 Author:lancea Date: 2013-08-12 16:09 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/cc64a05836a7 8022753: SQLXML javadoc example typo Reviewed-by: alanb, mchung ! src/share/classes/java/sql/SQLXML.java

Re: RFR [8022584] Memory leak in some NetworkInterface methods

2013-08-12 Thread David Holmes
Thanks Ivan. David On 12/08/2013 10:33 PM, Ivan Gerasimov wrote: David, Chris, I reverted back NULL-checking. Now the change consists of one line removal and a regression test. Webrev: http://cr.openjdk.java.net/~igerasim/8022584/6/webrev/ Hg export: http://cr.openjdk.java.net/~igerasim/2comm