hg: jdk8/tl/jdk: 6863624: java/util/Currency/PropertiesTest.sh writable check is incorrect

2013-06-21 Thread naoto . sato
Changeset: 8b84d557570c Author:naoto Date: 2013-06-21 13:42 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/8b84d557570c 6863624: java/util/Currency/PropertiesTest.sh writable check is incorrect Reviewed-by: alanb ! test/java/util/Currency/PropertiesTest.sh ! test/java/util/L

hg: jdk8/tl/jdk: 7131192: BigInteger.doubleValue() is depressingly slow

2013-06-21 Thread alan . bateman
Changeset: 814759462705 Author:bpb Date: 2013-06-21 11:50 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/814759462705 7131192: BigInteger.doubleValue() is depressingly slow Summary: In doubleValue() and floatValue() replace converting to String and parsing to Double or Float

hg: jdk8/tl/jdk: 7192954: Fix Float.parseFloat to round correctly and preserve monotonicity.; ...

2013-06-21 Thread alan . bateman
Changeset: a88f6f4d279f Author:bpb Date: 2013-06-21 11:12 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/a88f6f4d279f 7192954: Fix Float.parseFloat to round correctly and preserve monotonicity. 4396272: Parsing doubles fails to follow IEEE for largest decimal that should yie

Re: RFR 8017271: Crash may occur in java.net.DualStackPlainSocketImpl::initIDs due to unchecked values returned from JNI functions

2013-06-21 Thread Kurchi Hazra
Looks good to me too. On 6/21/2013 7:03 AM, Alan Bateman wrote: On 21/06/2013 14:50, Chris Hegarty wrote: There is a remote possibility that FindClass can return NULL, for a class we expect to exist, OOM, etc. Best practice is to check the return value before attempting to use it. CHECK_NULL

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

2013-06-21 Thread Chris Hegarty
On 21/06/2013 14:55, Andreas Rieber wrote: Hi Chris, actually from the HTTP spec any request on the server side should handle a request body even if not used or required by the request method. On You are correct Andreas. I meant 'harmless' from the stand-point of compatibility. Of course, we

Re: RFR 8017271: Crash may occur in java.net.DualStackPlainSocketImpl::initIDs due to unchecked values returned from JNI functions

2013-06-21 Thread Alan Bateman
On 21/06/2013 14:50, Chris Hegarty wrote: There is a remote possibility that FindClass can return NULL, for a class we expect to exist, OOM, etc. Best practice is to check the return value before attempting to use it. CHECK_NULL [1] is a macro used in other places in the networking native cod

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

2013-06-21 Thread Andreas Rieber
Hi Chris, actually from the HTTP spec any request on the server side should handle a request body even if not used or required by the request method. On the client side they say the TRACE method must not have one. For all others its kind of open. I remember that we used request body in GET and

RFR 8017271: Crash may occur in java.net.DualStackPlainSocketImpl::initIDs due to unchecked values returned from JNI functions

2013-06-21 Thread Chris Hegarty
There is a remote possibility that FindClass can return NULL, for a class we expect to exist, OOM, etc. Best practice is to check the return value before attempting to use it. CHECK_NULL [1] is a macro used in other places in the networking native code for such checks. There will be a pending

hg: jdk8/tl/jdk: 8001326: Improve Kerberos caching

2013-06-21 Thread weijun . wang
Changeset: 4503e04141f7 Author:weijun Date: 2013-06-21 18:26 +0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/4503e04141f7 8001326: Improve Kerberos caching Reviewed-by: valeriep ! src/share/classes/sun/security/jgss/krb5/AcceptSecContextToken.java ! src/share/classes/sun/secu

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

2013-06-21 Thread Chris Hegarty
Hi Andreas, Your changes look fine to me. Strange to include a body in a DELETE request, but seems harmless. I noticed that the javafx issue is no longer dependent on this, but still worth fixing. I can sponsor this change into jdk8 for you. -Chris. On 20/06/2013 23:04, Andreas Rieber wro