From 7427ba99987799d7b7e7910d11979903b6b3f3ea Mon Sep 17 00:00:00 2001
From: John-David Dalton <john.david.dalton@gmail.com>
Date: Sun, 4 Dec 2011 12:24:24 -0500
Subject: [PATCH] bean: Reduce memory load, and teardown slowdown by reusing
 handlers where possible. [jddalton]

---
 tests/benchmark.html | 56 +++++++++++++++++---------------------------
 1 file changed, 22 insertions(+), 34 deletions(-)

diff --git a/tests/benchmark.html b/tests/benchmark.html
index 18efc8b..9fb8132 100644
--- a/tests/benchmark.html
+++ b/tests/benchmark.html
@@ -16,7 +16,6 @@ <h1>Running Benchmark Suite</h1>
     <script>
 
       var element;
-      var fn = function () {};
       var suite = new Benchmark.Suite;
 
       suite
@@ -64,56 +63,42 @@ <h1>Running Benchmark Suite</h1>
               }) }
             , { 'setup': function () { element = document.createElement('div'); }
               , 'teardown': function () { bean.remove(element); }
-              })
+              }
+            )
 
         .add( 'remove()'
-            , function() { bean.remove( element ); }
-            , { 'setup': function () {
-                  var i = 100;
-                  element = document.createElement('div');
-                  while (i--) bean.add(element, 'click', function () {});
-                }
+            , function() {
+                bean.add(element, 'click', function () {});
+                bean.remove( element );
               }
             )
 
         .add( 'remove( event )'
-            , function() { bean.remove( element, 'click' ); }
-            , { 'setup': function () {
-                  var i = 100;
-                  element = document.createElement('div');
-                  while (i--) bean.add(element, 'click', function () {});
-                }
+            , function() {
+                bean.add(element, 'click', function () {});
+                bean.remove( element, 'click' );
               }
             )
 
         .add( 'remove( custom )'
-            , function() { bean.remove( element, 'fat' ); }
-            , { 'setup': function () {
-                  var i = 100;
-                  element = document.createElement('div');
-                  while (i--) bean.add(element, 'fat', function () {});
-                }
+            , function() {
+                bean.add(element, 'fat', function () {});
+                bean.remove( element, 'fat' );
               }
             )
 
         .add( 'remove( namespace )'
-            , function() { bean.remove( element, '.foo' ); }
-            , { 'setup': function () {
-                  var i = 100;
-                  element = document.createElement('div');
-                  while (i--) bean.add(element, 'click.foo', function () {});
-                }
+            , function() {
+                bean.add(element, 'click.foo', function () {});
+                bean.remove( element, 'click.foo' );
               }
             )
 
         .add( 'remove( event, fn )'
-            , function() { bean.remove( element, 'click', fn ); }
-            , { 'setup': function () {
-                  var i = 99;
-                  element = document.createElement('div');
-                  while (i--) bean.add(element, 'click', function () {});
-                  bean.add(element, 'click', fn);
-                }
+            , function() {
+                var fn = function () {};
+                bean.add(element, 'click', fn);
+                bean.remove( element, 'click', fn );
               }
             )
 
@@ -124,6 +109,7 @@ <h1>Running Benchmark Suite</h1>
                   element = document.createElement('div');
                   while (i--) bean.add(element, 'click', function () {});
                 }
+              , 'teardown': function () { bean.remove(element); }
               }
             )
 
@@ -134,16 +120,18 @@ <h1>Running Benchmark Suite</h1>
                   element = document.createElement('div');
                   while (i--) bean.add(element, 'fat', function () {});
                 }
+              , 'teardown': function () { bean.remove(element); }
               }
             )
 
         .add( 'fire( namespace )'
-            , function() { bean.fire( element, '.foo' ); }
+            , function() { bean.fire( element, 'click.foo' ); }
             , { 'setup': function () {
                   var i = 100;
                   element = document.createElement('div');
                   while (i--) bean.add(element, 'click.foo', function () {});
                 }
+              , 'teardown': function () { bean.remove(element); }
               }
             )