use traditional for loop instead of Array.forEach() for better perfor…

…mance
manidlou · Feb 17, 2017 · db694e5 · db694e5
1 parent e2ccf48
commit db694e5
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,8 @@
+1.1.2 / 2017-02-17
+------------------
+
+- Changed to traditional for loop instead of using Array.forEach() because of better performance.
+
 1.1.1 / 2017-02-05
 ------------------
 

diff --git a/klaw-sync.js b/klaw-sync.js
@@ -45,15 +45,15 @@ function walkSync (dir, opts, list) {
   } catch (er) {
     throw er
   }
-  files.forEach(function (file) {
+
+  for (var i = 0; i < files.length; i += 1) {
+    var file = files[i]
     if (ignore.length > 0) {
-      if (ignore.indexOf(file) === -1) {
-        _procPath(dir, file, opts, list)
-      }
+      if (ignore.indexOf(file) === -1) _procPath(dir, file, opts, list)
     } else {
       _procPath(dir, file, opts, list)
     }
-  })
+  }
   return list
 }
 

diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "klaw-sync",
   "version": "1.1.1",
-  "description": "Recursive and synchronous file system walker",
+  "description": "Recursive, synchronous, and fast file system walker",
   "main": "klaw-sync.js",
   "repository": {
     "type": "git",