diff options
author | Ben Noordhuis <info@bnoordhuis.nl> | 2017-11-16 15:15:05 +0100 |
---|---|---|
committer | Ben Noordhuis <info@bnoordhuis.nl> | 2017-11-17 02:37:55 +0100 |
commit | d6ac8a4db0c0a588258f594dc21fbd8018bef7c2 (patch) | |
tree | 46e336cedab7f85032efd524bdafcdd7e6cd92a3 | |
parent | d37789deeb5e951500d7fb499602dc100d3212fb (diff) | |
download | android-node-v8-d6ac8a4db0c0a588258f594dc21fbd8018bef7c2.tar.gz android-node-v8-d6ac8a4db0c0a588258f594dc21fbd8018bef7c2.tar.bz2 android-node-v8-d6ac8a4db0c0a588258f594dc21fbd8018bef7c2.zip |
src: use NODE_BUILTIN_MODULE_CONTEXT_AWARE() macro
Commit d217b2850e ("async_hooks: add trace events to async_hooks")
used `NODE_MODULE_CONTEXT_AWARE_BUILTIN()` instead.
After commit 8680bb9f1a ("src: explicitly register built-in modules")
it no longer works for static library builds so remove it.
PR-URL: https://github.com/nodejs/node/pull/17071
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
-rw-r--r-- | src/node.h | 7 | ||||
-rw-r--r-- | src/node_internals.h | 7 | ||||
-rw-r--r-- | src/node_trace_events.cc | 2 | ||||
-rw-r--r-- | test/cctest/node_module_reg.cc | 1 |
4 files changed, 9 insertions, 8 deletions
diff --git a/src/node.h b/src/node.h index 5f74854891..1d4583bbcc 100644 --- a/src/node.h +++ b/src/node.h @@ -442,10 +442,6 @@ typedef void (*addon_context_register_func)( v8::Local<v8::Context> context, void* priv); -#define NM_F_BUILTIN 0x01 -#define NM_F_LINKED 0x02 -#define NM_F_INTERNAL 0x04 - struct node_module { int nm_version; unsigned int nm_flags; @@ -529,9 +525,6 @@ extern "C" NODE_EXTERN void node_module_register(void* mod); #define NODE_MODULE_CONTEXT_AWARE(modname, regfunc) \ NODE_MODULE_CONTEXT_AWARE_X(modname, regfunc, NULL, 0) -#define NODE_MODULE_CONTEXT_AWARE_BUILTIN(modname, regfunc) \ - NODE_MODULE_CONTEXT_AWARE_X(modname, regfunc, NULL, NM_F_BUILTIN) \ - /* * For backward compatibility in add-on modules. */ diff --git a/src/node_internals.h b/src/node_internals.h index b432c44beb..b67a69b989 100644 --- a/src/node_internals.h +++ b/src/node_internals.h @@ -55,6 +55,12 @@ #define Z_MAX_LEVEL 9 #define Z_DEFAULT_LEVEL Z_DEFAULT_COMPRESSION +enum { + NM_F_BUILTIN = 1 << 0, + NM_F_LINKED = 1 << 1, + NM_F_INTERNAL = 1 << 2, +}; + struct sockaddr; // Variation on NODE_DEFINE_CONSTANT that sets a String value. @@ -115,6 +121,7 @@ struct sockaddr; V(stream_wrap) \ V(tcp_wrap) \ V(timer_wrap) \ + V(trace_events) \ V(tty_wrap) \ V(udp_wrap) \ V(url) \ diff --git a/src/node_trace_events.cc b/src/node_trace_events.cc index 20edb66cd6..b0ffe68eae 100644 --- a/src/node_trace_events.cc +++ b/src/node_trace_events.cc @@ -133,4 +133,4 @@ void InitializeTraceEvents(Local<Object> target, } // namespace node -NODE_MODULE_CONTEXT_AWARE_BUILTIN(trace_events, node::InitializeTraceEvents) +NODE_BUILTIN_MODULE_CONTEXT_AWARE(trace_events, node::InitializeTraceEvents) diff --git a/test/cctest/node_module_reg.cc b/test/cctest/node_module_reg.cc index f8d9d03c1c..a0736d2cc3 100644 --- a/test/cctest/node_module_reg.cc +++ b/test/cctest/node_module_reg.cc @@ -20,6 +20,7 @@ void _register_spawn_sync() {} void _register_stream_wrap() {} void _register_tcp_wrap() {} void _register_timer_wrap() {} +void _register_trace_events() {} void _register_tty_wrap() {} void _register_udp_wrap() {} void _register_util() {} |