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.
