Re: Code review request, 8030829 Add MD5 to jdk.certpath.disabledAlgorithms security property

2014-01-09 Thread Xuelei Fan
On 1/10/2014 6:34 AM, Sean Mullan wrote: The code change looks fine. My main concern is the number of tests that have been converted to run in othervm which will make the tests run slower. Did you explore how much effort it would be to convert some of the test certificates to use stronger algorit

Re: Code review request, 8030829 Add MD5 to jdk.certpath.disabledAlgorithms security property

2014-01-09 Thread Sean Mullan
The code change looks fine. My main concern is the number of tests that have been converted to run in othervm which will make the tests run slower. Did you explore how much effort it would be to convert some of the test certificates to use stronger algorithms? Also, I noticed many of the tests

hg: jdk8/tl/jdk: 8031187: DoubleStream.count is incorrect for a stream containing > Integer.MAX_VALUE elements

2014-01-09 Thread paul . sandoz
Changeset: 630a92015993 Author:psandoz Date: 2014-01-09 10:52 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/630a92015993 8031187: DoubleStream.count is incorrect for a stream containing > Integer.MAX_VALUE elements Reviewed-by: darcy ! src/share/classes/java/util/stream/Do