Cache array length in ol.Collection

This commit is contained in:
Tom Payne
2013-04-20 11:27:44 +02:00
parent f047e5f6bd
commit 332fd250b4
+6 -5
View File
@@ -87,8 +87,8 @@ ol.Collection.prototype.clear = function() {
* @param {Array} arr Array. * @param {Array} arr Array.
*/ */
ol.Collection.prototype.extend = function(arr) { ol.Collection.prototype.extend = function(arr) {
var i; var i, ii;
for (i = 0; i < arr.length; ++i) { for (i = 0, ii = arr.length; i < ii; ++i) {
this.push(arr[i]); this.push(arr[i]);
} }
}; };
@@ -165,9 +165,10 @@ ol.Collection.prototype.push = function(elem) {
* @return {*} The removed element or undefined if elem was not found. * @return {*} The removed element or undefined if elem was not found.
*/ */
ol.Collection.prototype.remove = function(elem) { ol.Collection.prototype.remove = function(elem) {
var i; var arr = this.array_;
for (i = 0; i < this.array_.length; ++i) { var i, ii;
if (this.array_[i] === elem) { for (i = 0, ii = arr.length; i < ii; ++i) {
if (arr[i] === elem) {
return this.removeAt(i); return this.removeAt(i);
} }
} }