OpenLayers OpenLayers

Changeset 7627

Show
Ignore:
Timestamp:
07/31/08 00:44:25 (4 months ago)
Author:
tschaub
Message:

Caching array length instead of accessing it with each iteration. r=crschmidt (closes #1636)

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/openlayers/lib/OpenLayers.js

    r7590 r7627  
    4141          
    4242            var scripts = document.getElementsByTagName('script'); 
    43             for (var i = 0; i < scripts.length; i++) { 
     43            for (var i=0, len=scripts.length; i<len; i++) { 
    4444                var src = scripts[i].getAttribute('src'); 
    4545                if (src) { 
    …… 
    223223        } 
    224224        var host = OpenLayers._getScriptLocation() + "lib/";     
    225         for (var i = 0; i < jsfiles.length; i++) { 
     225        for (var i=0, len=jsfiles.length; i<len; i++) { 
    226226            if (docWrite) { 
    227227                allScriptTags[i] = "<script src='" + host + jsfiles[i] + 

  • trunk/openlayers/lib/OpenLayers/BaseTypes.js

    r7451 r7627  
    8585        var oStringList = str.split('-'); 
    8686        var camelizedString = oStringList[0]; 
    87         for (var i = 1; i < oStringList.length; i++) { 
     87        for (var i=1, len=oStringList.length; i<len; i++) { 
    8888            var s = oStringList[i]; 
    8989            camelizedString += s.charAt(0).toUpperCase() + s.substring(1); 
    …… 
    119119        var tokens = template.split("${"); 
    120120        var item, last, replacement; 
    121         for(var i=1; i<tokens.length; i++) { 
     121        for(var i=1, len=tokens.length; i<len; i++) { 
    122122            item = tokens[i]; 
    123123            last = item.indexOf("}");  

  • trunk/openlayers/lib/OpenLayers/BaseTypes/Class.js

    r6131 r7627  
    3535    var extended = {}; 
    3636    var parent; 
    37     for(var i=0; i<arguments.length; ++i) { 
     37    for(var i=0, len=arguments.length; i<len; ++i) { 
    3838        if(typeof arguments[i] == "function") { 
    3939            // get the prototype of the superclass 
    …… 
    8686    var superClass = arguments[0]; 
    8787    var proto = new superClass(OpenLayers.Class.isPrototype); 
    88     for (var i = 1; i < arguments.length; i++) { 
     88    for (var i=1, len=arguments.length; i<len; i++) { 
    8989        if (typeof arguments[i] == "function") { 
    9090            var mixin = arguments[i]; 

  • trunk/openlayers/lib/OpenLayers/BaseTypes/Element.js

    r7579 r7627  
    2929     */ 
    3030    toggle: function() { 
    31         for (var i = 0; i < arguments.length; i++) { 
     31        for (var i=0, len=arguments.length; i<len; i++) { 
    3232            var element = OpenLayers.Util.getElement(arguments[i]); 
    3333            var display = OpenLayers.Element.visible(element) ? 'hide'  
    …… 
    4646     */ 
    4747    hide: function() { 
    48         for (var i = 0; i < arguments.length; i++) { 
     48        for (var i=0, len=arguments.length; i<len; i++) { 
    4949            var element = OpenLayers.Util.getElement(arguments[i]); 
    5050            element.style.display = 'none'; 
    …… 
    6060     */ 
    6161    show: function() { 
    62         for (var i = 0; i < arguments.length; i++) { 
     62        for (var i=0, len=arguments.length; i<len; i++) { 
    6363            var element = OpenLayers.Util.getElement(arguments[i]); 
    6464            element.style.display = ''; 

  • trunk/openlayers/lib/OpenLayers/Console.js

    r6131 r7627  
    221221    if(window.console) { 
    222222        var scripts = document.getElementsByTagName("script"); 
    223         for(var i=0; i<scripts.length; ++i) { 
     223        for(var i=0, len=scripts.length; i<len; ++i) { 
    224224            if(scripts[i].src.indexOf("firebug.js") != -1) { 
    225225                OpenLayers.Util.extend(OpenLayers.Console, console); 

  • trunk/openlayers/lib/OpenLayers/Control/ArgParser.js

    r5900 r7627  
    6969 
    7070        //make sure we dont already have an arg parser attached 
    71         for(var i=0; i< this.map.controls.length; i++) { 
     71        for(var i=0, len=this.map.controls.length; i<len; i++) { 
    7272            var control = this.map.controls[i]; 
    7373            if ( (control != this) && 
    …… 
    142142            this.map.events.unregister('addlayer', this, this.configureLayers); 
    143143 
    144             for(var i=0; i < this.layers.length; i++) { 
     144            for(var i=0, len=this.layers.length; i<len; i++) { 
    145145                 
    146146                var layer = this.map.layers[i]; 

  • trunk/openlayers/lib/OpenLayers/Control/Attribution.js

    r6149 r7627  
    7676        var attributions = []; 
    7777        if (this.map && this.map.layers) { 
    78             for(var i=0; i < this.map.layers.length; i++) { 
     78            for(var i=0, len=this.map.layers.length; i<len; i++) { 
    7979                var layer = this.map.layers[i]; 
    8080                if (layer.attribution && layer.getVisibility()) { 

  • trunk/openlayers/lib/OpenLayers/Control/EditingToolbar.js

    r5614 r7627  
    3737          new OpenLayers.Control.DrawFeature(layer, OpenLayers.Handler.Polygon, {'displayClass': 'olControlDrawFeaturePolygon'}) 
    3838        ]; 
    39         for (var i = 0; i < controls.length; i++) { 
     39        for (var i=0, len=controls.length; i<len; i++) { 
    4040            controls[i].featureAdded = function(feature) { feature.state = OpenLayers.State.INSERT; }; 
    4141        } 

  • trunk/openlayers/lib/OpenLayers/Control/LayerSwitcher.js

    r7382 r7627  
    178178        var layers = this[layersType + "Layers"]; 
    179179        if (layers) { 
    180             for(var i=0; i < layers.length; i++) { 
     180            for(var i=0, len=layers.length; i<len ; i++) { 
    181181                var layer = layers[i]; 
    182182                OpenLayers.Event.stopObservingElement(layer.inputElem); 
    …… 
    202202            redraw = true; 
    203203        } else { 
    204             for (var i=0; i < this.layerStates.length; i++) { 
     204            for (var i=0, len=this.layerStates.length; i<len; i++) { 
    205205                var layerState = this.layerStates[i]; 
    206206                var layer = this.map.layers[i]; 
    …… 
    244244        // we will trigger more visibility changes, and we want to not redraw 
    245245        // and enter an infinite loop. 
    246         this.layerStates = new Array(this.map.layers.length); 
    247         for (var i = 0; i < this.map.layers.length; i++) { 
     246        var len = this.map.layers.length; 
     247        this.layerStates = new Array(len); 
     248        for (var i=0; i <len; i++) { 
    248249            var layer = this.map.layers[i]; 
    249250            this.layerStates[i] = { 
    …… 
    257258        var layers = this.map.layers.slice(); 
    258259        if (!this.ascending) { layers.reverse(); } 
    259         for( var i = 0; i < layers.length; i++) { 
     260        for(var i=0, len=layers.length; i<len; i++) { 
    260261            var layer = layers[i]; 
    261262            var baseLayer = layer.isBaseLayer; 
    …… 
    388389 
    389390        // set the newly selected base layer         
    390         for(var i=0; i < this.baseLayers.length; i++) { 
     391        for(var i=0, len=this.baseLayers.length; i<len; i++) { 
    391392            var layerEntry = this.baseLayers[i]; 
    392393            if (layerEntry.inputElem.checked) { 
    …… 
    396397 
    397398        // set the correct visibilities for the overlays 
    398         for(var i=0; i < this.dataLayers.length; i++) { 
     399        for(var i=0, len=this.dataLayers.length; i<len; i++) { 
    399400            var layerEntry = this.dataLayers[i];    
    400401            layerEntry.layer.setVisibility(layerEntry.inputElem.checked); 

  • trunk/openlayers/lib/OpenLayers/Control/ModifyFeature.js

    r7616 r7627  
    549549        var control = this; 
    550550        function collectComponentVertices(geometry) { 
    551             var i, vertex, component; 
     551            var i, vertex, component, len; 
    552552            if(geometry.CLASS_NAME == "OpenLayers.Geometry.Point") { 
    553553                vertex = new OpenLayers.Feature.Vector(geometry); 
    …… 
    570570                // add virtual vertices in the middle of each edge 
    571571                if(geometry.CLASS_NAME != "OpenLayers.Geometry.MultiPoint") { 
    572                     for(i=0; i<geometry.components.length-1; ++i) { 
     572                    for(i=0, len=geometry.components.length; i<len-1; ++i) { 
    573573                        var prevVertex = geometry.components[i]; 
    574574                        var nextVertex = geometry.components[i + 1]; 

  • trunk/openlayers/lib/OpenLayers/Control/MouseToolbar.js

    r6149 r7627  
    272272            } 
    273273            if (this.mode == "measure" && mode != "measure") { 
    274                 for(var i = 0; i < this.measureDivs.length; i++) { 
     274                for(var i=0, len=this.measureDivs.length; i<len; i++) { 
    275275                    if (this.measureDivs[i]) {  
    276276                        this.map.layerContainerDiv.removeChild(this.measureDivs[i]); 

  • trunk/openlayers/lib/OpenLayers/Control/OverviewMap.js

    r7374 r7627  
    241241            var eventsToStop = ['dblclick','mousedown']; 
    242242             
    243             for (var i = 0; i < eventsToStop.length; i++) { 
     243            for (var i=0, len=eventsToStop.length; i<len; i++) { 
    244244 
    245245                OpenLayers.Event.observe(this.maximizeDiv,  

  • trunk/openlayers/lib/OpenLayers/Control/Panel.js

    r7597 r7627  
    6363    activate: function() { 
    6464        if (OpenLayers.Control.prototype.activate.apply(this, arguments)) { 
    65             for(var i = 0; i < this.controls.length; i++) { 
     65            for(var i=0, len=this.controls.length; i<len; i++) { 
    6666                if (this.controls[i] == this.defaultControl) { 
    6767                    this.controls[i].activate(); 
    …… 
    8080    deactivate: function() { 
    8181        if (OpenLayers.Control.prototype.deactivate.apply(this, arguments)) { 
    82             for(var i = 0; i < this.controls.length; i++) { 
     82            for(var i=0, len=this.controls.length; i<len; i++) { 
    8383                this.controls[i].deactivate(); 
    8484            }     
    …… 
    9797    draw: function() { 
    9898        OpenLayers.Control.prototype.draw.apply(this, arguments); 
    99         for (var i = 0; i < this.controls.length; i++) { 
     99        for (var i=0, len=this.controls.length; i<len; i++) { 
    100100            this.map.addControl(this.controls[i]); 
    101101            this.controls[i].deactivate(); 
    …… 
    116116        this.div.innerHTML = ""; 
    117117        if (this.active) { 
    118             for (var i = 0; i < this.controls.length; i++) { 
     118            for (var i=0, len=this.controls.length; i<len; i++) { 
    119119                var element = this.controls[i].panel_div; 
    120120                if (this.controls[i].active) { 
    …… 
    150150            return; 
    151151        } 
    152         for (var i = 0; i < this.controls.length; i++) { 
     152        for (var i=0, len=this.controls.length; i<len; i++) { 
    153153            if (this.controls[i] != control) { 
    154154                if (this.controls[i].type != OpenLayers.Control.TYPE_TOGGLE) { 
    …… 
    180180        // Also, stop mousedowns and clicks, but don't stop mouseup, 
    181181        // since they need to pass through. 
    182         for (var i = 0; i < controls.length; i++) { 
     182        for (var i=0, len=controls.length; i<len; i++) { 
    183183            var element = document.createElement("div"); 
    184184            var textNode = document.createTextNode(" "); 
    …… 
    194194 
    195195        if (this.map) { // map.addControl() has already been called on the panel 
    196             for (var i = 0; i < controls.length; i++) { 
     196            for (var i=0, len=controls.length; i<len; i++) { 
    197197                this.map.addControl(controls[i]); 
    198198                controls[i].deactivate(); 

  • trunk/openlayers/lib/OpenLayers/Control/Permalink.js

    r7396 r7627  
    8080 
    8181        //make sure we have an arg parser attached 
    82         for(var i=0; i< this.map.controls.length; i++) { 
     82        for(var i=0, len=this.map.controls.length; i<len; i++) { 
    8383            var control = this.map.controls[i]; 
    8484            if (control.CLASS_NAME == "OpenLayers.Control.ArgParser") { 
    …… 
    154154         
    155155        params.layers = ''; 
    156         for (var i=0; i < this.map.layers.length; i++) { 
     156        for (var i=0, len=this.map.layers.length; i<len; i++) { 
    157157            var layer = this.map.layers[i]; 
    158158 

  • trunk/openlayers/lib/OpenLayers/Events.js

    r7615 r7627  
    450450        this.eventTypes = []; 
    451451        if (eventTypes != null) { 
    452             for (var i = 0; i < eventTypes.length; i++) { 
     452            for (var i=0, len=eventTypes.length; i<len; i++) { 
    453453                this.addEventType(eventTypes[i]); 
    454454            } 
    …… 
    500500     */ 
    501501    attachToElement: function (element) { 
    502         for (var i = 0; i < this.BROWSER_EVENTS.length; i++) { 
     502        for (var i=0, len=this.BROWSER_EVENTS.length; i<len; i++) { 
    503503            var eventType = this.BROWSER_EVENTS[i]; 
    504504 
    …… 
    640640        var listeners = this.listeners[type]; 
    641641        if (listeners != null) { 
    642             for (var i = 0; i < listeners.length; i++) { 
     642            for (var i=0, len=listeners.length; i<len; i++) { 
    643643                if (listeners[i].obj == obj && listeners[i].func == func) { 
    644644                    listeners.splice(i, 1); 
    …… 
    694694        if ((listeners != null) && (listeners.length > 0)) { 
    695695            var continueChain; 
    696             for (var i = 0; i < listeners.length; i++) { 
     696            for (var i=0, len=listeners.length; i<len; i++) { 
    697697                var callback = listeners[i]; 
    698698                // bind the context to callback.obj 

  • trunk/openlayers/lib/OpenLayers/Filter/FeatureId.js

    r6833 r7627  
    5454     */ 
    5555    evaluate: function(feature) { 
    56         for (var i=0; i<this.fids.length; i++) { 
     56        for (var i=0, len=this.fids.length; i<len; i++) { 
    5757            var fid = feature.fid || feature.id; 
    5858            if (fid == this.fids[i]) { 

  • trunk/openlayers/lib/OpenLayers/Filter/Logical.js

    r6818 r7627  
    7171        switch(this.type) { 
    7272            case OpenLayers.Filter.Logical.AND: 
    73                 for (var i=0; i<this.filters.length; i++) { 
     73                for (var i=0, len=this.filters.length; i<len; i++) { 
    7474                    if (this.filters[i].evaluate(context) == false) { 
    7575                        return false; 
    …… 
    7979                 
    8080            case OpenLayers.Filter.Logical.OR: 
    81                 for (var i=0; i<this.filters.length; i++) { 
     81                for (var i=0, len=this.filters.length; i<len; i++) { 
    8282                    if (this.filters[i].evaluate(context) == true) { 
    8383                        return true; 

  • trunk/openlayers/lib/OpenLayers/Format/GPX.js

    r7590 r7627  
    8585        if(this.extractTracks) { 
    8686            var tracks = doc.getElementsByTagName("trk"); 
    87             for (var i = 0, len = tracks.length; i < tracks.length; i++) { 
     87            for (var i=0, len=tracks.length; i<len; i++) { 
    8888                // Attributes are only in trk nodes, not trkseg nodes 
    8989                var attrs = {} 
    …… 
    104104        if(this.extractRoutes) { 
    105105            var routes = doc.getElementsByTagName("rte"); 
    106             for (var k = 0; k < routes.length; k++) { 
     106            for (var k=0, klen=routes.length; k<klen; k++) { 
    107107                var attrs = {} 
    108108                if(this.extractAttributes) { 

  • trunk/openlayers/lib/OpenLayers/Format/GeoJSON.js

    r6131 r7627  
    105105                            break; 
    106106                        case "FeatureCollection": 
    107                             for(var i=0; i<obj.features.length; ++i) { 
     107                            for(var i=0, len=obj.features.length; i<len; ++i) { 
    108108                                try { 
    109109                                    results.push(this.parseFeature(obj.features[i])); 
    …… 
    279279            var points = []; 
    280280            var p = null; 
    281             for(var i=0; i<array.length; ++i) { 
     281            for(var i=0, len=array.length; i<len; ++i) { 
    282282                try { 
    283283                    p = this.parseCoords["point"].apply(this, [array[i]]); 
    …… 
    304304            var points = []; 
    305305            var p = null; 
    306             for(var i=0; i<array.length; ++i) { 
     306            for(var i=0, len=array.length; i<len; ++i) { 
    307307                try { 
    308308                    p = this.parseCoords["point"].apply(this, [array[i]]); 
    …… 
    329329            var lines = []; 
    330330            var l = null; 
    331             for(var i=0; i<array.length; ++i) { 
     331            for(var i=0, len=array.length; i<len; ++i) { 
    332332                try { 
    333333                    l = this.parseCoords["linestring"].apply(this, [array[i]]); 
    …… 
    351351            var rings = []; 
    352352            var r, l; 
    353             for(var i=0; i<array.length; ++i) { 
     353            for(var i=0, len=array.length; i<len; ++i) { 
    354354                try { 
    355355                    l = this.parseCoords["linestring"].apply(this, [array[i]]); 
    …… 
    377377            var polys = []; 
    378378            var p = null; 
    379             for(var i=0; i<array.length; ++i) { 
     379            for(var i=0, len=array.length; i<len; ++i) { 
    380380                try { 
    381381                    p = this.parseCoords["polygon"].apply(this, [array[i]]); 
    …… 
    583583        'multipoint': function(multipoint) { 
    584584            var array = []; 
    585             for(var i=0; i<multipoint.components.length; ++i) { 
     585            for(var i=0, len=multipoint.components.length; i<len; ++i) { 
    586586                array.push(this.extract.point.apply(this, [multipoint.components[i]])); 
    587587            } 
    …… 
    602602        'linestring': function(linestring) { 
    603603            var array = []; 
    604             for(var i=0; i<linestring.components.length; ++i) { 
     604            for(var i=0, len=linestring.components.length; i<len; ++i) { 
    605605                array.push(this.extract.point.apply(this, [linestring.components[i]])); 
    606606            } 
    …… 
    621621        'multilinestring': function(multilinestring) { 
    622622            var array = []; 
    623             for(var i=0; i<multilinestring.components.length; ++i) { 
     623            for(var i=0, len=multilinestring.components.length; i<len; ++i) { 
    624624                array.push(this.extract.linestring.apply(this, [multilinestring.components[i]])); 
    625625            } 
    …… 
    639639        'polygon': function(polygon) { 
    640640            var array = []; 
    641             for(var i=0; i<polygon.components.length; ++i) { 
     641            for(var i=0, len=polygon.components.length; i<len; ++i) { 
    642642                array.push(this.extract.linestring.apply(this, [polygon.components[i]])); 
    643643            } 
    …… 
    658658        'multipolygon': function(multipolygon) { 
    659659            var array = []; 
    660             for(var i=0; i<multipolygon.components.length; ++i) { 
     660            for(var i=0, len=multipolygon.components.length; i<len; ++i) { 
    661661                array.push(this.extract.polygon.apply(this, [multipolygon.components[i]])); 
    662662            } 

  • trunk/openlayers/lib/OpenLayers/Format/GeoRSS.js

    r7468 r7627  
    132132            var components = [];  
    133133            var point; 
    134             for (var i=0; i < coords.length; i+=2) { 
     134            for (var i=0, len=coords.length; i<len; i+=2) { 
    135135                point = new OpenLayers.Geometry.Point(parseFloat(coords[i+1]),  
    136136                                                     parseFloat(coords[i])); 
    …… 
    142142            var components = [];  
    143143            var point; 
    144             for (var i=0; i < coords.length; i+=2) { 
     144            for (var i=0, len=coords.length; i<len; i+=2) { 
    145145                point = new OpenLayers.Geometry.Point(parseFloat(coords[i+1]),  
    146146                                                     parseFloat(coords[i])); 
    …… 
    276276        if(features instanceof Array) { 
    277277            georss = this.createElementNS(this.rssns, "rss"); 
    278             for(var i=0; i < features.length; i++) { 
     278            for(var i=0, len=features.length; i<len; i++) { 
    279279                georss.appendChild(this.createFeatureXML(features[i])); 
    280280            } 

  • trunk/openlayers/lib/OpenLayers/Format/JSON.js

    r6801 r7627  
    267267            this.level += 1; 
    268268     
    269             for(var i=0; i<array.length; ++i) { 
     269            for(var i=0, len=array.length; i<len; ++i) { 
    270270                // recursive calls need to allow for sub-classing 
    271271                json = OpenLayers.Format.JSON.prototype.write.apply(this, 

  • trunk/openlayers/lib/OpenLayers/Format/KML.js

    r7607 r7627  
    168168        // process the nodes found  
    169169        var types = ["Link", "NetworkLink", "Style", "StyleMap", "Placemark"]; 
    170         for(var i=0; i<types.length; ++i) { 
     170        for(var i=0, len=types.length; i<len; ++i) { 
    171171            var type = types[i]; 
    172172 
    …… 
    229229        newOptions.depth++; 
    230230 
    231         for(var i=0; i < nodes.length; i++) { 
     231        for(var i=0, len=nodes.length; i<len; i++) { 
    232232            var href = this.parseProperty(nodes[i], "*", "href"); 
    233233            if(href && !this.fetched[href]) { 
    …… 
    268268     */ 
    269269    parseStyles: function(nodes, options) { 
    270         for(var i=0; i < nodes.length; i++) { 
     270        for(var i=0, len=nodes.length; i<len; i++) { 
    271271            var style = this.parseStyle(nodes[i]); 
    272272            if(style) { 
    …… 
    292292        var types = ["LineStyle", "PolyStyle", "IconStyle", "BalloonStyle"]; 
    293293        var type, nodeList, geometry, parser; 
    294         for(var i=0; i<types.length; ++i) { 
     294        for(var i=0, len=types.length; i<len; ++i) { 
    295295            type = types[i]; 
    296296            styleTypeNode = this.getElementsByTagNameNS(node,  
    …… 
    490490        // To do the select or "highlight" bit, we'd need to change lots more 
    491491 
    492         for(var i=0; i < nodes.length; i++) { 
     492        for(var i=0, len=nodes.length; i<len; i++) { 
    493493            var node = nodes[i]; 
    494494            var pairs = this.getElementsByTagNameNS(node, "*",  
    …… 
    496496 
    497497            var id = node.getAttribute("id"); 
    498             for (var j=0; j<pairs.length; j++) { 
     498            for (var j=0, jlen=pairs.length; j<jlen; j++) { 
    499499                var pair = pairs[j]; 
    500500                // Use the shortcut in the SLD format to quickly retrieve the  
    …… 
    531531    parseFeatures: function(nodes, options) { 
    532532        var features = new Array(nodes.length); 
    533         for(var i=0; i < nodes.length; i++) { 
     533        for(var i=0, len=nodes.length; i<len; i++) { 
    534534            var featureNode = nodes[i]; 
    535535            var feature = this.parseFeature.apply(this,[featureNode]) ; 
    …… 
    585585        var order = ["MultiGeometry", "Polygon", "LineString", "Point"]; 
    586586        var type, nodeList, geometry, parser; 
    587         for(var i=0; i<order.length; ++i) { 
     587        for(var i=0, len=order.length; i<len; ++i) { 
    588588            type = order[i]; 
    589589            this.internalns = node.namespaceURI ?  
    …… 
    779779                // this assumes exterior ring first, inner rings after 
    780780                var ring; 
    781                 for(var i=0; i<nodeList.length; ++i) { 
     781                for(var i=0, len=nodeList.length; i<len; ++i) { 
    782782                    ring = this.parseGeometry.linestring.apply(this, 
    783783                                                        [nodeList[i], true]); 
    …… 
    807807            var parts = []; 
    808808            var children = node.childNodes; 
    809             for(var i=0; i<children.length; ++i ) { 
     809            for(var i=0, len=children.length; i<len; ++i ) { 
    810810                child = children[i]; 
    811811                if(child.nodeType == 1) { 
    …… 
    838838        var child, grandchildren, grandchild; 
    839839        var children = node.childNodes; 
    840         for(var i=0; i<children.length; ++i) { 
     840        for(var i=0, len=children.length; i<len; ++i) { 
    841841            child = children[i]; 
    842842            if(child.nodeType == 1) { 
    …… 
    910910        var kml = this.createElementNS(this.kmlns, "kml"); 
    911911        var folder = this.createFolderXML(); 
    912         for(var i=0; i<features.length; ++i) { 
     912        for(var i=0, len=features.length; i<len; ++i) { 
    913913            folder.appendChild(this.createPlacemarkXML(features[i])); 
    914914        } 
    …… 
    11091109            var rings = geometry.components; 
    11101110            var ringMember, ringGeom, type; 
    1111             for(var i=0; i<rings.length; ++i) { 
     1111            for(var i=0, len=rings.length; i<len; ++i) { 
    11121112                type = (i==0) ? "outerBoundaryIs" : "innerBoundaryIs"; 
    11131113                ringMember = this.createElementNS(this.kmlns, type); 
    …… 
    11481148            var kml = this.createElementNS(this.kmlns, "MultiGeometry"); 
    11491149            var child; 
    1150             for(var i=0; i<geometry.components.length; ++i) { 
     1150            for(var i=0, len=geometry.components.length; i<len; ++i) { 
    11511151                child = this.buildGeometryNode.apply(this, 
    11521152                                                     [geometry.components[i]]); 

  • trunk/openlayers/lib/OpenLayers/Format/SLD/v1.js

    r7333 r7627  
    116116                this.readChildNodes(node, layer); 
    117117                // give each of the user styles this layer name 
    118                 for(var i=0; i<layer.userStyles.length; ++i) { 
     118                for(var i=0, len=layer.userStyles.length; i<len; ++i) { 
    119119                    layer.userStyles[i].layerName = layer.name; 
    120120                } 
    …… 
    211211                ]; 
    212212                var prop, value; 
    213                 for(var i=0; i<properties.length; ++i) { 
     213                for(var i=0, len=properties.length; i<len; ++i) { 
    214214                    prop = properties[i]; 
    215215                    value = graphic[prop]; 
    …… 
    563563                // add in named styles 
    564564                if(layer.namedStyles) { 
    565                     for(var i=0; i<layer.namedStyles.length; ++i) { 
     565                    for(var i=0, len=layer.namedStyles.length; i<len; ++i) { 
    566566                        this.writeNode( 
    567567                            node, "NamedStyle", layer.namedStyles[i] 
    …… 
    572572                // add in user styles 
    573573                if(layer.userStyles) { 
    574                     for(var i=0; i<layer.userStyles.length; ++i) { 
     574                    for(var i=0, len=layer.userStyles.length; i<len; ++i) { 
    575575                        this.writeNode( 
    576576                            node, "UserStyle", layer.userStyles[i] 
    …… 
    625625                 
    626626                // add in rules 
    627                 for(var i=0; i<style.rules.length; ++i) { 
     627                for(var i=0, len=style.rules.length; i<len; ++i) { 
    628628                    this.writeNode(node, "Rule", style.rules[i]); 
    629629                } 
    …… 
    671671                var types = OpenLayers.Style.SYMBOLIZER_PREFIXES; 
    672672                var type, symbolizer; 
    673                 for(var i=0; i<types.length; ++i) { 
     673                for(var i=0, len=types.length; i<len; ++i) { 
    674674                    type = types[i]; 
    675675                    symbolizer = rule.symbolizer[type]; 
    …… 
    777777                node.appendChild(this.createTextNode(tokens[0])); 
    778778                var item, last; 
    779                 for(var i=1; i<tokens.length; i++) { 
     779                for(var i=1, len=tokens.length; i<len; i++) { 
    780780                    item = tokens[i]; 
    781781                    last = item.indexOf("}");  
    …… 
    903903                var sub = filter.CLASS_NAME.split(".").pop(); 
    904904                if(sub == "FeatureId") { 
    905                     for(var i=0; i<filter.fids.length; ++i) { 
     905                    for(var i=0, len=filter.fids.length; i<len; ++i) { 
    906906                        this.writeNode(node, "FeatureId", filter.fids[i]); 
    907907                    } 
    …… 
    919919                var node = this.createElementNSPlus("ogc:And"); 
    920920                var childFilter; 
    921                 for(var i=0; i<filter.filters.length; ++i) { 
     921                for(var i=0, len=filter.filters.length; i<len; ++i) { 
    922922                    childFilter = filter.filters[i]; 
    923923                    this.writeNode( 
    …… 
    930930                var node = this.createElementNSPlus("ogc:Or"); 
    931931                var childFilter; 
    932                 for(var i=0; i<filter.filters.length; ++i) { 
     932                for(var i=0, len=filter.filters.length; i<len; ++i) { 
    933933                    childFilter = filter.filters[i]; 
    934934                    this.writeNode( 
    …… 
    11041104        var children = node.childNodes; 
    11051105        var child, group, reader, prefix, local; 
    1106         for(var i=0; i<children.length; ++i) { 
     1106        for(var i=0, len=children.length; i<len; ++i) { 
    11071107            child = children[i]; 
    11081108            if(child.nodeType == 1) { 

  • trunk/openlayers/lib/OpenLayers/Format/WKT.js

    r7413 r7627  
    7171                           type != "geometrycollection" && 
    7272                           typeof features == "object") { 
    73                     for (var i = 0; i < features.length; i++) { 
     73                    for (var i=0, len=features.length; i<len; i++) { 
    7474