Skip to content

Commit 71faa5c

Browse files
BridgeARevanlucas
authored andcommitted
benchmark: (events) use destructuring
PR-URL: #18250 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent c25d4d6 commit 71faa5c

6 files changed

+6
-20
lines changed

benchmark/events/ee-add-remove.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,7 @@ const events = require('events');
44

55
const bench = common.createBenchmark(main, { n: [25e4] });
66

7-
function main(conf) {
8-
const n = conf.n | 0;
9-
7+
function main({ n }) {
108
const ee = new events.EventEmitter();
119
const listeners = [];
1210

benchmark/events/ee-emit.js

+1-5
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,7 @@ const bench = common.createBenchmark(main, {
88
listeners: [1, 5, 10],
99
});
1010

11-
function main(conf) {
12-
const n = conf.n | 0;
13-
const argc = conf.argc | 0;
14-
const listeners = Math.max(conf.listeners | 0, 1);
15-
11+
function main({ n, argc, listeners }) {
1612
const ee = new EventEmitter();
1713

1814
for (var k = 0; k < listeners; k += 1)

benchmark/events/ee-listener-count-on-prototype.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,7 @@ const EventEmitter = require('events').EventEmitter;
44

55
const bench = common.createBenchmark(main, { n: [5e7] });
66

7-
function main(conf) {
8-
const n = conf.n | 0;
9-
7+
function main({ n }) {
108
const ee = new EventEmitter();
119

1210
for (var k = 0; k < 5; k += 1) {

benchmark/events/ee-listeners-many.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,7 @@ const EventEmitter = require('events').EventEmitter;
44

55
const bench = common.createBenchmark(main, { n: [5e6] });
66

7-
function main(conf) {
8-
const n = conf.n | 0;
9-
7+
function main({ n }) {
108
const ee = new EventEmitter();
119
ee.setMaxListeners(101);
1210

benchmark/events/ee-listeners.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,7 @@ const EventEmitter = require('events').EventEmitter;
44

55
const bench = common.createBenchmark(main, { n: [5e6] });
66

7-
function main(conf) {
8-
const n = conf.n | 0;
9-
7+
function main({ n }) {
108
const ee = new EventEmitter();
119

1210
for (var k = 0; k < 5; k += 1) {

benchmark/events/ee-once.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,7 @@ const EventEmitter = require('events').EventEmitter;
44

55
const bench = common.createBenchmark(main, { n: [2e7] });
66

7-
function main(conf) {
8-
const n = conf.n | 0;
9-
7+
function main({ n }) {
108
const ee = new EventEmitter();
119

1210
function listener() {}

0 commit comments

Comments
 (0)