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]

Reply via email to