Review request: Reduce size of Collisions.java test run

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