Don't use for..in for n array
authorThiefMaster <adrian@planetcoding.net>
Thu, 16 May 2013 20:45:54 +0000 (22:45 +0200)
committerThiefMaster <adrian@planetcoding.net>
Thu, 16 May 2013 20:45:54 +0000 (22:45 +0200)
Every time you use for..in for an Array, Justin Beiber kills a kitten.

server/configuration.js

index 02961ca8b22412265a9c10fdeffac75371869a29..be89037208f9904afab89b042382a5549710831e 100644 (file)
@@ -8,10 +8,11 @@ var config_filename = 'config.js',
 
 function loadConfig() {
     var new_config,
-        conf_filepath;
+        conf_filepath,
+        i;
 
     // Loop through the possible config paths and find a usable one
-    for (var i in config_dirs) {
+    for (i = 0; i < config_dirs.length; i++) {
         conf_filepath = config_dirs[i] + config_filename;
 
         try {
@@ -57,4 +58,4 @@ module.exports.get = function (specific_environment) {
     return loaded_config[specific_environment] || {};
 };
 
-module.exports.loadConfig = loadConfig;
\ No newline at end of file
+module.exports.loadConfig = loadConfig;