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