process: refactor execution

• Removed unreachable code of the `evalModule()` function as an early error is thrown when the `print` parameter is a truthy value. • Make use of the nullish coalescing operator. PR-URL: #40664 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Minwoo Jung <[email protected]>
nodejs · Nov 25, 2021 · 58de6ce · 58de6ce
1 parent 1c8590e
commit 58de6ce
Showing 1 changed file with 2 additions and 8 deletions.
diff --git a/lib/internal/process/execution.js b/lib/internal/process/execution.js
@@ -42,15 +42,9 @@ function evalModule(source, print) {
   if (print) {
     throw new ERR_EVAL_ESM_CANNOT_PRINT();
   }
-  const { log } = require('internal/console/global');
   const { loadESM } = require('internal/process/esm_loader');
   const { handleMainPromise } = require('internal/modules/run_main');
-  return handleMainPromise(loadESM(async (loader) => {
-    const { result } = await loader.eval(source);
-    if (print) {
-      log(result);
-    }
-  }));
+  return handleMainPromise(loadESM((loader) => loader.eval(source)));
 }
 
 function evalScript(name, body, breakFirstLine, print) {
@@ -158,7 +152,7 @@ function createOnGlobalUncaughtException() {
               'Exception',
             'Exception',
             null,
-            er ? er : {});
+            er ?? {});
         }
       } catch {}  // Ignore the exception. Diagnostic reporting is unavailable.
     }