1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: remove try/catch in common.isMainThread

Browse Source 
Refactor common.isMainThread.

PR-URL: https://github.com/nodejs/node/pull/25249
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This commit is contained in:
Rich Trott 2018-12-28 08:55:38 -08:00
parent 39a2ac4c6b
commit 9c2b14013b
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
test/common/index.js
View File

@ -39,12 +39,11 @@ const noop = () => {};
const hasCrypto = Boolean(process.versions.openssl); const hasCrypto = Boolean(process.versions.openssl);
const isMainThread = (() => { const isMainThread = (() => {
try { if (require('module').builtinModules.includes('worker_threads')) {
return require('worker_threads').isMainThread; return require('worker_threads').isMainThread;
} catch {
// Worker module not enabled → only a single main thread exists.
return true;
} }
// Worker module not enabled → only a single main thread exists.
return true;
})(); })();
// Check for flags. Skip this for workers (both, the `cluster` module and // Check for flags. Skip this for workers (both, the `cluster` module and