58a65d6689Instead of always checking whether we've already warned about a possible EventEmitter memory leak, first run the rest of the code as accessing random properties on an Array is expensive. In addition, remove an unnecessary truthy check. PR-URL: https://github.com/nodejs/node/pull/20452 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
26 lines
644 B
JavaScript
26 lines
644 B
JavaScript
'use strict';
|
|
const common = require('../common.js');
|
|
const events = require('events');
|
|
|
|
const bench = common.createBenchmark(main, { n: [1e6] });
|
|
|
|
function main({ n }) {
|
|
const ee = new events.EventEmitter();
|
|
const listeners = [];
|
|
|
|
var k;
|
|
for (k = 0; k < 10; k += 1)
|
|
listeners.push(function() {});
|
|
|
|
bench.start();
|
|
for (var i = 0; i < n; i += 1) {
|
|
const dummy = (i % 2 === 0) ? 'dummy0' : 'dummy1';
|
|
for (k = listeners.length; --k >= 0; /* empty */) {
|
|
ee.on(dummy, listeners[k]);
|
|
}
|
|
for (k = listeners.length; --k >= 0; /* empty */) {
|
|
ee.removeListener(dummy, listeners[k]);
|
|
}
|
|
}
|
|
bench.end(n);
|
|
}
|