remove forEach + closure overhead

emberjs · Mar 16, 2017 · 96c147b · 96c147b
1 parent 13bbb8c
commit 96c147b
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/addon/-private/system/record-array-manager.js b/addon/-private/system/record-array-manager.js
@@ -105,18 +105,18 @@ export default class RecordArrayManager {
     this._pending = Object.create(null);
     let modelsToRemove = [];
 
-    Object.keys(pending).forEach(modelName => {
+    for (let modelName in pending) {
       let internalModels = pending[modelName];
-
-      internalModels.forEach(internalModel => {
+      for (let j = 0; j < internalModels.length; j++) {
+        let internalModel = internalModels[j];
         // mark internalModels, so they can once again be processed by the
         // recordArrayManager
         internalModel._pendingRecordArrayManagerFlush = false;
         // build up a set of models to ensure we have purged correctly;
         if (internalModel.isHiddenFromRecordArrays()) {
           modelsToRemove.push(internalModel);
         }
-      });
+      }
 
       // process filteredRecordArrays
       if (this._filteredRecordArrays[modelName]) {
@@ -136,7 +136,7 @@ export default class RecordArrayManager {
       if (modelsToRemove.length > 0) {
         this.removeFromAdapterPopulatedRecordArrays(modelsToRemove);
       }
-    });
+    }
   }
 
   updateLiveRecordArray(modelName, internalModels) {