On 24 January 2011 13:03, Gilles Sadowski <gil...@harfang.homelinux.org> wrote: > Hi. > > I've updated the unit test file "FastMathTestPerformance.java" in > revision 1062761. > If there are no complaints with the new setup, I'll remove the method > "testPerformance" (currently set as "@Ignore").
Yes, separate tests are better. However, I think we should use nanoTime for the time measurement, and the results could be presented as multiples of the base value. Absolute times are not particularly relevant for the comparisons within a method, though may be useful to know the comparisons between different methods. So I suggest showing only one absolute time - perhaps the Math one? - and displaying the others as multiples of it. This would probably be easiest to do with a helper method for formatting the results. > > Best, > Gilles > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
