Proper oom calculation

isaacs · Sep 29, 2010 · d14aa70 · d14aa70
1 parent 4022216
commit d14aa70
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/lib/bench.js b/lib/bench.js
@@ -60,8 +60,6 @@ function run (fn, time, countPerLap, cb) {
     lap();
   };
   runner();
-
-  return p;
 };
 
 function compare (set, compareCount, time, countPerLap, cb) {
@@ -168,12 +166,14 @@ function pct (num1, num2) {
 function times (num1, num2) {
   return Math.round((Math.max(num1, num2)/Math.min(num1, num2)) * 100) / 100;
 }
+function logX (n, x) {
+  return Math.log(n)/Math.log(x)
+}
 function oom (num1, num2) {
-  var ratio = Math.max(num1, num2)/Math.min(num1, num2),
+  var l1 = logX(num1, 10)
-    oom = 0;
+    , l2 = logX(num2, 10)
-  // poor man's logarithm
+    , diff = Math.abs( l1 - l2 )
-  while ( (ratio/=10) > 0.9 ) oom++;
+  return Math.round(diff * 100) / 100
-  return oom;
 }
 function avg (nums) {
   var sum = 0;