Simplify EventEmitterEx.emit with ...args

nodules · Jun 15, 2018 · ef29547 · ef29547
1 parent 58c36d3
commit ef29547
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/lib/event_emitter_ex.js b/lib/event_emitter_ex.js
@@ -14,14 +14,12 @@ function inspect(val) {
 
 // add 'luster:eex' to the `NODE_DEBUG` environment variable to enable events logging
 if (process.env.NODE_DEBUG && /luster:eex/i.test(process.env.NODE_DEBUG)) {
-    EventEmitterEx.prototype.emit = function() {
-        const args = Array.prototype
-            .slice.call(arguments, 0)
-            .map(inspect);
+    EventEmitterEx.prototype.emit = function(...args) {
+        const inspectedArgs = args.map(inspect).join(', ');
 
-        console.log('%s(%s).emit(%s)', this.constructor.name || 'EventEmitterEx', this.wid, args.join(', '));
+        console.log('%s(%s).emit(%s)', this.constructor.name || 'EventEmitterEx', this.wid, inspectedArgs);
 
-        return EventEmitter.prototype.emit.apply(this, arguments);
+        return EventEmitter.prototype.emit.apply(this, args);
     };
 }