console: apply null as this for util.format

Util.format is just a stateless function. Apply current console as `this` is unnecessary. PR-URL: #5222 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Roman Reiss <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
nodejs · Mar 1, 2016 · de4c07b · de4c07b
1 parent 81d4127
commit de4c07b
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/console.js b/lib/console.js
@@ -33,15 +33,15 @@ function Console(stdout, stderr) {
 }
 
 Console.prototype.log = function() {
-  this._stdout.write(util.format.apply(this, arguments) + '\n');
+  this._stdout.write(util.format.apply(null, arguments) + '\n');
 };
 
 
 Console.prototype.info = Console.prototype.log;
 
 
 Console.prototype.warn = function() {
-  this._stderr.write(util.format.apply(this, arguments) + '\n');
+  this._stderr.write(util.format.apply(null, arguments) + '\n');
 };
 
 
@@ -75,7 +75,7 @@ Console.prototype.trace = function trace() {
   // exposed.
   var err = new Error();
   err.name = 'Trace';
-  err.message = util.format.apply(this, arguments);
+  err.message = util.format.apply(null, arguments);
   Error.captureStackTrace(err, trace);
   this.error(err.stack);
 };
@@ -84,7 +84,7 @@ Console.prototype.trace = function trace() {
 Console.prototype.assert = function(expression) {
   if (!expression) {
     var arr = Array.prototype.slice.call(arguments, 1);
-    require('assert').ok(false, util.format.apply(this, arr));
+    require('assert').ok(false, util.format.apply(null, arr));
   }
 };