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 --- node.gyp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'node.gyp') diff --git a/node.gyp b/node.gyp index 3621cc5e9d..8560c1d512 100644 --- a/node.gyp +++ b/node.gyp @@ -325,7 +325,6 @@ 'sources': [ 'src/async_wrap.cc', - 'src/bootstrapper.cc', 'src/callback_scope.cc', 'src/cares_wrap.cc', 'src/connect_wrap.cc', @@ -372,6 +371,7 @@ 'src/node_serdes.cc', 'src/node_stat_watcher.cc', 'src/node_symbols.cc', + 'src/node_task_queue.cc', 'src/node_trace_events.cc', 'src/node_types.cc', 'src/node_url.cc', -- cgit v1.2.3