Use in-place filtering for pre-render functions

2013-01-20 21:56:27 +01:00
parent d76db2005f
commit 3967e17366
1 changed files with 9 additions and 6 deletions
--- a/src/ol/map.js
+++ b/src/ol/map.js
@@ -641,12 +641,15 @@ ol.Map.prototype.renderFrame_ = function(time) {
    };
  }

-  this.preRenderFunctions_ = goog.array.filter(
-      this.preRenderFunctions_,
-      function(preRenderFunction) {
-        return preRenderFunction(this, frameState);
-      },
-      this);
+  var preRenderFunctions = this.preRenderFunctions_;
+  var n = 0, preRenderFunction;
+  for (i = 0; i < preRenderFunctions.length; ++i) {
+    preRenderFunction = preRenderFunctions[i];
+    if (preRenderFunction(this, frameState)) {
+      preRenderFunctions[n++] = preRenderFunction;
+    }
+  }
+  preRenderFunctions.length = n;

  if (!goog.isNull(frameState)) {
    // FIXME works for View2D only