Review request: Reduce size of Collisions.java test run (original) (raw)

Mike Duigou mike.duigou at oracle.com
Fri Sep 14 19:20:34 UTC 2012

I have updated the webrev to make these changes. I also added an @run with the alternative hashing behaviour enabled. This should have been added when the behaviour was made optional.

New webrev:

http://cr.openjdk.java.net/~mduigou/7189926/1/webrev/

Mike

On Sep 11 2012, at 08:01 , Alan Bateman wrote:

On 11/09/2012 15:50, Mike Duigou wrote:

Hello All;

A very small patch to the Collisions.java regression test which reduces the size of the test by 50% in the default case. We've run into problems with this test taking too long on small or older platforms. http://cr.openjdk.java.net/~mduigou/7189926/0/webrev/ Thanks, Mike This looks okay to me. I guess I probably would s/KEYS/keys/ as it is no longer a constant, probably s/MAPS/maps/ too but it's minor observation. -Alan.

More information about the core-libs-dev mailing list