node: deprecate process.EventEmitter

The comment stating it was deprecated was added in 2011 via
4ef8f06. It is time to
actually deprecate it.

PR-URL: #5049
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>

Author: evanlucas

src/node.js

@@ -20,7 +20,17 @@
     });
     EventEmitter.call(process);
 
-    process.EventEmitter = EventEmitter; // process.EventEmitter is deprecated
+    let eeWarned = false;
+    Object.defineProperty(process, 'EventEmitter', {
+      get() {
+        const internalUtil = NativeModule.require('internal/util');
+        eeWarned = internalUtil.printDeprecationMessage(
+          `process.EventEmitter is deprecated. Use require('events') instead.`,
+          eeWarned
+        );
+        return EventEmitter;
+      }
+    });
 
     startup.setupProcessObject();