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.