summaryrefslogtreecommitdiffstats
path: root/make
diff options
context:
space:
mode:
authorElijah C. Menifee <[email protected]>2010-08-23 03:02:43 +0800
committerSven Gothel <[email protected]>2010-11-06 03:49:25 +0100
commit4c808ec572808a221d0ce08785dad2d18f77ea5f (patch)
tree1505e48475d97d3a8422aaf335fee75edd6b922d /make
parent0674c42c0b18073b53a09300d0f60373f1463665 (diff)
Fixed benchmarking bugs in primitive HashMap test.
Both IntIntHashMapTest and LongIntHashMapTest failed to reset test start time. The time values for intmapGetTime,mapGetTime,intmapRemoveTime,and mapRemoveTime all used the start time value from mapPutTime. This caused the following to be always/guarenteed to be true: mapPutTime < intmapGetTime < mapGetTime < intmapRemoveTime < mapRemoveTime thus not actually testing the following asserts: assertTrue("'get' too slow", intmapGetTime <= mapGetTime); assertTrue("'remove' too slow", intmapRemoveTime <= mapRemoveTime); In addition the results of the test were being printed before the actual test ran. Changed code to reset time to System.nanoTime() prior to each for loop, and moved time calculation and result output to below the test for loop. I also improved the output of the test info, by including the warmup status on benchmark start.
Diffstat (limited to 'make')
0 files changed, 0 insertions, 0 deletions