Avoid allocating some closures for iteration

Addepar · May 23, 2019 · 92d3fed · 92d3fed
1 parent 204c4a3
commit 92d3fed
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/addon/components/-private/row-wrapper.js b/addon/components/-private/row-wrapper.js
@@ -43,7 +43,9 @@ export default Component.extend({
   },
 
   destroy() {
-    this._cells.forEach(cell => cell.destroy());
+    for (let cell in this._cells) {
+      cell.destroy();
+    }
 
     this._super(...arguments);
   },
@@ -94,7 +96,8 @@ export default Component.extend({
         }
       }
 
-      _cells.forEach((cell, i) => {
+      for (let i = 0; i < this._cells.length; i++) {
+        let cell = this._cells[i];
         let columnValue = objectAt(columns, i);
         let columnMeta = this.get('columnMetaCache').get(columnValue);
 
@@ -107,7 +110,7 @@ export default Component.extend({
           rowSelectionMode,
           rowValue,
         });
-      });
+      }
 
       return _cells;
     }