[fixup] more flexible emit benchmark

nodejs · Nov 8, 2017 · 8b95f82 · 8b95f82
1 parent b659275
commit 8b95f82
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 43 deletions.
diff --git a/benchmark/events/ee-emit-2-args.js b/benchmark/events/ee-emit-2-args.js
diff --git a/benchmark/events/ee-emit-4-args.js b/benchmark/events/ee-emit-4-args.js
diff --git a/benchmark/events/ee-emit.js b/benchmark/events/ee-emit.js
@@ -2,19 +2,29 @@
 const common = require('../common.js');
 const EventEmitter = require('events').EventEmitter;
 
-const bench = common.createBenchmark(main, { n: [2e6] });
+const bench = common.createBenchmark(main, {
+  n: [2e6],
+  argc: [0, 2, 4, 10],
+  listeners: [1, 5, 10],
+});
 
 function main(conf) {
   const n = conf.n | 0;
+  const argc = conf.argc | 0;
+  const listeners = conf.listeners | 0;
 
   const ee = new EventEmitter();
 
-  for (var k = 0; k < 10; k += 1)
+  const args = new Array(argc);
+  args.fill(10);
+  args.unshift('dummy');
+
+  for (var k = 0; k < listeners; k += 1)
     ee.on('dummy', function() {});
 
   bench.start();
   for (var i = 0; i < n; i += 1) {
-    ee.emit('dummy');
+    ee.emit.apply(ee, args);
   }
   bench.end(n);
 }