let keyword makes it scoped already (winstonjs#1436)

* let keyword makes it scoped already * miss * another miss (squash)
Mizumaki · Aug 20, 2018 · 9719ed4 · 9719ed4
1 parent de245e6
commit 9719ed4
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/winston/transports/file.js b/lib/winston/transports/file.js
@@ -594,8 +594,8 @@ module.exports = class File extends TransportStream {
 
     // const isZipped = this.zippedArchive ? '.gz' : '';
     const isZipped = this.zippedArchive ? '.gz' : '';
-    for (let x = this.maxFiles - 1; x > 0; x--) {
-      tasks.push(function (i) {
+    for (let i = this.maxFiles - 1; i > 0; i--) {
+      tasks.push(() => {
         return cb => {
           let fileName = `${basename}${(i - 1)}${ext}${isZipped}`;
           const tmppath = path.join(this.dirname, fileName);
@@ -609,7 +609,7 @@ module.exports = class File extends TransportStream {
             fs.rename(tmppath, path.join(this.dirname, fileName), cb);
           });
         };
-      }.bind(this, x));
+      });
     }
 
     asyncSeries(tasks, () => {