events: fix duplicate require which cause performance penalty

nodejs · Aug 26, 2021 · 56a674f · 56a674f
1 parent 31772a4
commit 56a674f
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/lib/events.js b/lib/events.js
@@ -63,9 +63,7 @@ const {
   },
 } = require('internal/errors');
 
-const {
-  inspect
-} = require('internal/util/inspect');
+let inspect;
 
 const {
   validateAbortSignal,
@@ -372,7 +370,9 @@ EventEmitter.prototype.emit = function emit(type, ...args) {
     }
 
     let stringifiedEr;
-    const { inspect } = require('internal/util/inspect');
+    if (!inspect) {
+      inspect = require('internal/util/inspect').inspect;
+    }
     try {
       stringifiedEr = inspect(er);
     } catch {
@@ -464,6 +464,9 @@ function _addListener(target, type, listener, prepend) {
     m = _getMaxListeners(target);
     if (m > 0 && existing.length > m && !existing.warned) {
       existing.warned = true;
+      if (!inspect) {
+        inspect = require('internal/util/inspect').inspect;
+      }
       // No error code for this since it is a Warning
       // eslint-disable-next-line no-restricted-syntax
       const w = new Error('Possible EventEmitter memory leak detected. ' +