diff options
| author | Ben Noordhuis <info@bnoordhuis.nl> | 2020-01-07 16:18:56 +0300 |
|---|---|---|
| committer | Myles Borins <mylesborins@google.com> | 2020-01-16 09:38:53 +0300 |
| commit | 6050236c3dfa22fd152bfa62e008e8db0d558e1f (patch) | |
| tree | fdd07b12695153f6639c0ee6e5f837a1a560f646 /src/node.cc | |
| parent | ff60a0e2b15428ab427e15afb9eeb573039c216d (diff) | |
src: remove uses of node::InitializeV8Platform()
This requires minor changes to src/env.cc to deal with
`node::tracing::AgentWriterHandle::GetTracingController()` now possibly
returning a nullptr, because the cctest doesn't set one.
It seems plausible to me that embedders won't set one either so that
seems like an okay change to make. It avoids embedders having to track
down nullptr segfaults.
PR-URL: https://github.com/nodejs/node/pull/31245
Refs: https://github.com/nodejs/node/pull/31217
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Diffstat (limited to 'src/node.cc')
| -rw-r--r-- | src/node.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/node.cc b/src/node.cc index e48827d0cb2..461bdc7f666 100644 --- a/src/node.cc +++ b/src/node.cc @@ -1077,7 +1077,8 @@ InitializationResult InitializeOncePerProcess(int argc, char** argv) { V8::SetEntropySource(crypto::EntropySource); #endif // HAVE_OPENSSL - InitializeV8Platform(per_process::cli_options->v8_thread_pool_size); + per_process::v8_platform.Initialize( + per_process::cli_options->v8_thread_pool_size); V8::Initialize(); performance::performance_v8_start = PERFORMANCE_NOW(); per_process::v8_initialized = true; |