node: set process._eventsCount to 0 on startup

process is an EventEmitter. There are operations that increment and decrement the _eventsCount property of an EventEmitter. process._eventsCount would previously get set to NaN. This change makes process._eventsCount be calculated as expected. PR-URL: #5208 Reviewed-By: Brian White <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
nodejs · Mar 1, 2016 · 78d6488 · 78d6488
1 parent 4a6e2b2
commit 78d6488
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/src/node.js b/src/node.js
@@ -12,6 +12,7 @@
 
   function startup() {
     var EventEmitter = NativeModule.require('events');
+    process._eventsCount = 0;
 
     Object.setPrototypeOf(process, Object.create(EventEmitter.prototype, {
       constructor: {

diff --git a/test/parallel/test-process-emit.js b/test/parallel/test-process-emit.js
@@ -18,3 +18,5 @@ process.on('SIGPIPE', common.mustCall((data) => {
 process.emit('normal', 'normalData');
 process.emit(sym, 'symbolData');
 process.emit('SIGPIPE', 'signalData');
+
+assert.strictEqual(isNaN(process._eventsCount), false);