Merge pull request #36 from doug-martin/master

v0.1.3
noolsjs · May 25, 2013 · 49bc7e3 · 49bc7e3
2 parents ff8d99b + 905b0d5
commit 49bc7e3
Show file tree

Hide file tree

Showing 6 changed files with 828 additions and 788 deletions.
diff --git a/docs/History.html b/docs/History.html
@@ -176,6 +176,10 @@
 
 
 
+<h1>v0.1.3 / 2012-05-24</h1>
+<ul>
+<li>Fixed memory leak with FactHashes for the agendas</li>
+</ul>
 <h1>v0.1.2 / 2012-05-24</h1>
 <ul>
 <li>Removed unneeded calls to <code>.then</code> to address performance degredation (#30)</li>

diff --git a/history.md b/history.md
@@ -1,3 +1,7 @@
+#v0.1.3 / 2012-05-24
+
+* Fixed memory leak with FactHashes for the agendas
+
 #v0.1.2 / 2012-05-24
 
 * Removed unneeded calls to `.then` to address performance degredation (#30)

diff --git a/lib/index.js b/lib/index.js
@@ -74,13 +74,29 @@ var FactHash = declare({
             return this.memoryValues[indexOf(this.memory, k)];
         },
 
+        __compact: function () {
+            var oldM = this.memory.slice(0),
+                oldMv = this.memoryValues.slice(0),
+                l = oldMv.length,
+                m = this.memory = [],
+                mv = this.memoryValues = [],
+                oldMemoryValue;
+            for (var i = 0; i < l; i++) {
+                oldMemoryValue = oldMv[i];
+                if (oldMemoryValue.length !== 0) {
+                    mv[m.push(oldM[i]) - 1] = oldMemoryValue;
+                }
+            }
+        },
+
         remove: function (v) {
             var facts = v.match.facts, j = facts.length - 1, mv = this.memoryValues, m = this.memory;
             for (; j >= 0; j--) {
                 var i = indexOf(m, facts[j]);
                 var arr = mv[i], index = indexOf(arr, v);
                 arr.splice(index, 1);
             }
+            this.__compact();
         },
 
         insert: function (insert) {