Author: burton Date: Thu Mar 3 12:48:20 2005 New Revision: 156081 URL: http://svn.apache.org/viewcvs?view=rev&rev=156081 Log: better dox...
Modified: jakarta/commons/sandbox/benchmark/trunk/src/java/org/apache/commons/benchmark/Benchmark.java jakarta/commons/sandbox/benchmark/trunk/src/java/org/apache/commons/benchmark/BenchmarkTracker.java Modified: jakarta/commons/sandbox/benchmark/trunk/src/java/org/apache/commons/benchmark/Benchmark.java URL: http://svn.apache.org/viewcvs/jakarta/commons/sandbox/benchmark/trunk/src/java/org/apache/commons/benchmark/Benchmark.java?view=diff&r1=156080&r2=156081 ============================================================================== --- jakarta/commons/sandbox/benchmark/trunk/src/java/org/apache/commons/benchmark/Benchmark.java (original) +++ jakarta/commons/sandbox/benchmark/trunk/src/java/org/apache/commons/benchmark/Benchmark.java Thu Mar 3 12:48:20 2005 @@ -189,6 +189,8 @@ */ public static Benchmark getBenchmark( String name ) { + //we have to synchronize on this hashmap I'm afraid. I could use a + //ConcurrentHashMap but I'm not sure of the performance advantage here. synchronized( benchmarks ) { Benchmark benchmark = (Benchmark)benchmarks.get( name ); @@ -221,7 +223,7 @@ //simple testing framework - INTERVAL = 10 * 1000; + INTERVAL = 10 * 1000; Benchmark benchmark = getBenchmark( Benchmark.class ); benchmark = benchmark.child( "main" ); Modified: jakarta/commons/sandbox/benchmark/trunk/src/java/org/apache/commons/benchmark/BenchmarkTracker.java URL: http://svn.apache.org/viewcvs/jakarta/commons/sandbox/benchmark/trunk/src/java/org/apache/commons/benchmark/BenchmarkTracker.java?view=diff&r1=156080&r2=156081 ============================================================================== --- jakarta/commons/sandbox/benchmark/trunk/src/java/org/apache/commons/benchmark/BenchmarkTracker.java (original) +++ jakarta/commons/sandbox/benchmark/trunk/src/java/org/apache/commons/benchmark/BenchmarkTracker.java Thu Mar 3 12:48:20 2005 @@ -37,6 +37,9 @@ */ private Benchmark parent = null; + /** + * Interval (in millis) that we rollover this tracked benchmark. + */ private long interval = 0; /** --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]