Revises loops on arrays

src/litegraph.js

@@ -189,7 +189,7 @@
 
 			//used to know which nodes create when dragging files to the canvas
             if (base_class.supported_extensions) {
-                for (var i in base_class.supported_extensions) {
+                for (var i=0; i < base_class.supported_extensions.length; i++) {
 					var ext = base_class.supported_extensions[i];
 					if(ext && ext.constructor === String)
 	                    this.node_types_by_file_extension[ ext.toLowerCase() ] = base_class;
@@ -401,14 +401,14 @@
             var tmp = document.getElementsByTagName("script");
             //weird, this array changes by its own, so we use a copy
             var script_files = [];
-            for (var i in tmp) {
+            for (var i=0; i < tmp.length; i++) {
                 script_files.push(tmp[i]);
             }
 
             var docHeadObj = document.getElementsByTagName("head")[0];
             folder_wildcard = document.location.href + folder_wildcard;
 
-            for (var i in script_files) {
+            for (var i=0; i < script_files.length; i++) {
                 var src = script_files[i].src;
                 if (
                     !src ||
@@ -1829,8 +1829,8 @@
                     "weird LLink bug, link info is not a LLink but a regular object"
                 );
                 var link2 = new LLink();
-                for (var i in link) {
-                    link2[i] = link[i];
+                for (var j in link) { 
+                    link2[j] = link[j];
                 }
                 this.links[i] = link2;
                 link = link2;
@@ -8618,7 +8618,7 @@ LGraphNode.prototype.executeAction = function(action)
 
         var values = LiteGraph.getNodeTypesCategories( canvas.filter );
         var entries = [];
-        for (var i in values) {
+        for (var i=0; i < values.length; i++) {
             if (values[i]) {
                 entries.push({ value: values[i], content: values[i], has_submenu: true });
             }
@@ -8631,7 +8631,7 @@ LGraphNode.prototype.executeAction = function(action)
             var category = v.value;
             var node_types = LiteGraph.getNodeTypesInCategory( category, canvas.filter );
             var values = [];
-            for (var i in node_types) {
+            for (var i=0; i < node_types.length; i++) {
                 if (!node_types[i].skip_list) {
                     values.push({
                         content: node_types[i].title,
@@ -8684,7 +8684,7 @@ LGraphNode.prototype.executeAction = function(action)
 
         var entries = [];
         if (options) {
-            for (var i in options) {
+            for (var i=0; i < options.length; i++) {
                 var entry = options[i];
                 if (!entry) {
                     entries.push(null);
@@ -8761,7 +8761,7 @@ LGraphNode.prototype.executeAction = function(action)
 
         var entries = [];
         if (options) {
-            for (var i in options) {
+            for (var i=0; i < options.length; i++) {
                 var entry = options[i];
                 if (!entry) {
                     //separator?
@@ -10326,7 +10326,7 @@ LGraphNode.prototype.executeAction = function(action)
 
         //entries
         var num = 0;
-        for (var i in values) {
+        for (var i=0; i < values.length; i++) {
             var name = values.constructor == Array ? values[i] : i;
             if (name != null && name.constructor !== String) {
                 name = name.content === undefined ? String(name) : name.content;
@@ -10616,7 +10616,7 @@ LGraphNode.prototype.executeAction = function(action)
             result.push(elements[i]);
         }
 
-        for (var i in result) {
+        for (var i=0; i < result.length; i++) {
             if (result[i].close) {
                 result[i].close();
             } else if (result[i].parentNode) {

src/nodes/graphics.js

@@ -240,7 +240,7 @@
 	c[2] = Math.abs( Math.sin( 0.01 * reModular.getTime() * Math.PI) );
 	*/
 
-        for (var i in result) {
+        for (var i=0; i < result.length; i++) {
             result[i] /= 255;
         }
 

src/nodes/interface.js

@@ -673,7 +673,7 @@
 
         if (typeof this.str == "string") {
             var lines = this.str.split("\\n");
-            for (var i in lines) {
+            for (var i=0; i < lines.length; i++) {
                 ctx.fillText(
                     lines[i],
                     this.properties["align"] == "left" ? 15 : this.size[0] - 15,
@@ -704,7 +704,7 @@
         this.last_ctx.font =
             this.properties["fontsize"] + "px " + this.properties["font"];
         var max = 0;
-        for (var i in lines) {
+        for (var i=0; i < lines.length; i++) {
             var w = this.last_ctx.measureText(lines[i]).width;
             if (max < w) {
                 max = w;

utils/server.js

@@ -7,4 +7,4 @@ app.use('/external', express.static('external'))
 app.use('/demo', express.static('demo'))
 app.use('/', express.static('demo'))
 
-app.listen(80, () => console.log('Example app listening on port 80!'))
+app.listen(80, () => console.log('Example app listening on port 80!'))