Using regexp literal and .map instead of a loop for replacing

BlueCocoa · BlueCocoa · commit 40b5f09fe8b7 · 2020-05-07T16:23:20.000+08:00
Signed-off-by: Cocoa &lt;0xbbc@0xbbc.com&gt;
diff --git a/lib/appenders/file.js b/lib/appenders/file.js
@@ -55,12 +55,8 @@ function fileAppender(file, layout, logSize, numBackups, options, timezoneOffset
   const app = function (loggingEvent) {
     if (options.removeColor === true) {
       // eslint-disable-next-line no-control-regex
-      const regex = new RegExp("\x1b[[0-9;]*m", "g");
-      for (let i = 0; i < loggingEvent.data.length; i += 1) {
-        let d = loggingEvent.data[i];
-        d = d.replace(regex, '');
-        loggingEvent.data[i] = d;
-      }
+      const regex = /\x1b[[0-9;]*m/g;
+      loggingEvent.data = loggingEvent.data.map(d => d.replace(regex, ''))
     }
     if (!writer.write(layout(loggingEvent, timezoneOffset) + eol, "utf8")) {
       process.emit('log4js:pause', true);
