From 457603e96194b4858ad715f9faacb3ad7fec7f35 Mon Sep 17 00:00:00 2001 From: Joyee Cheung Date: Wed, 19 Dec 2018 18:44:14 +0800 Subject: src: move process.nextTick and promise setup into node_task_queue.cc This patch: - Moves the process.nextTick and promise setup C++ code into node_task_queue.cc which is exposed as `internalBinding('task_queue')` - Makes `lib/internal/process/promises.js` and `lib/internal/process/next_tick.js` as side-effect-free as possible - Removes the bootstrapper object being passed into `bootstrap/node.js`, let `next_tick.js` and `promises.js` load whatever they need from `internalBinding('task_queue')` instead. - Rename `process._tickCallback` to `runNextTicks` internally for clarity but still expose it as `process._tickCallback`. PR-URL: https://github.com/nodejs/node/pull/25163 Refs: https://github.com/nodejs/node/issues/24961 Reviewed-By: Ruben Bridgewater Reviewed-By: Anatoli Papirovski --- src/node_internals.h | 2 -- 1 file changed, 2 deletions(-) (limited to 'src/node_internals.h') diff --git a/src/node_internals.h b/src/node_internals.h index 99498a6218..897905b5a7 100644 --- a/src/node_internals.h +++ b/src/node_internals.h @@ -185,8 +185,6 @@ v8::Maybe ProcessEmitDeprecationWarning(Environment* env, const char* warning, const char* deprecation_code); -void SetupBootstrapObject(Environment* env, - v8::Local bootstrapper); void SetupProcessObject(Environment* env, const std::vector& args, const std::vector& exec_args); -- cgit v1.2.3