From 7c6ccca57289acedfe579d2bc32998e5752f0815 Mon Sep 17 00:00:00 2001 From: Frederic Junod Date: Wed, 17 Dec 2014 10:35:07 +0100 Subject: [PATCH] Reuse previousExtent_ local variable --- src/ol/map.js | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/ol/map.js b/src/ol/map.js index 4ee6642c5f..ba42b80d8b 100644 --- a/src/ol/map.js +++ b/src/ol/map.js @@ -215,7 +215,7 @@ ol.Map = function(options) { * @private * @type {ol.Extent} */ - this.previousExtent_ = null; + this.previousExtent_ = ol.extent.createEmpty(); /** * @private @@ -1261,13 +1261,12 @@ ol.Map.prototype.renderFrame_ = function(time) { var idle = this.preRenderFunctions_.length === 0 && !frameState.viewHints[ol.ViewHint.ANIMATING] && !frameState.viewHints[ol.ViewHint.INTERACTING] && - (!this.previousExtent_ || - !ol.extent.equals(frameState.extent, this.previousExtent_)); + !ol.extent.equals(frameState.extent, this.previousExtent_); if (idle) { this.dispatchEvent( new ol.MapEvent(ol.MapEventType.MOVEEND, this, frameState)); - this.previousExtent_ = ol.extent.clone(frameState.extent); + ol.extent.clone(frameState.extent, this.previousExtent_); } }