src: fix type mismatch warnings from missing priv

Registration initialization functions are expected to have a 4th argument, a void*, so add them where necessary to fix the warnings. PR-URL: nodejs#24737 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
refack · Jan 10, 2019 · f16e1cf · f16e1cf
1 parent a5ad13f
commit f16e1cf
Show file tree

Hide file tree

Showing 43 changed files with 96 additions and 45 deletions.
diff --git a/src/async_wrap.cc b/src/async_wrap.cc
@@ -440,7 +440,8 @@ Local<FunctionTemplate> AsyncWrap::GetConstructorTemplate(Environment* env) {
 
 void AsyncWrap::Initialize(Local<Object> target,
                            Local<Value> unused,
-                           Local<Context> context) {
+                           Local<Context> context,
+                           void* priv) {
   Environment* env = Environment::GetCurrent(context);
   Isolate* isolate = env->isolate();
   HandleScope scope(isolate);

diff --git a/src/async_wrap.h b/src/async_wrap.h
@@ -116,7 +116,8 @@ class AsyncWrap : public BaseObject {
 
   static void Initialize(v8::Local<v8::Object> target,
                          v8::Local<v8::Value> unused,
-                         v8::Local<v8::Context> context);
+                         v8::Local<v8::Context> context,
+                         void* priv);
 
   static void GetAsyncId(const v8::FunctionCallbackInfo<v8::Value>& args);
   static void PushAsyncIds(const v8::FunctionCallbackInfo<v8::Value>& args);

diff --git a/src/bootstrapper.cc b/src/bootstrapper.cc
@@ -171,7 +171,8 @@ namespace symbols {
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
 #define V(PropertyName, StringValue)                                        \
     target->Set(env->context(),                                             \

diff --git a/src/cares_wrap.cc b/src/cares_wrap.cc
@@ -2172,7 +2172,8 @@ void StrError(const FunctionCallbackInfo<Value>& args) {
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   env->SetMethod(target, "getaddrinfo", GetAddrInfo);

diff --git a/src/fs_event_wrap.cc b/src/fs_event_wrap.cc
@@ -52,7 +52,8 @@ class FSEventWrap: public HandleWrap {
  public:
   static void Initialize(Local<Object> target,
                          Local<Value> unused,
-                         Local<Context> context);
+                         Local<Context> context,
+                         void* priv);
   static void New(const FunctionCallbackInfo<Value>& args);
   static void Start(const FunctionCallbackInfo<Value>& args);
   static void GetInitialized(const FunctionCallbackInfo<Value>& args);
@@ -95,7 +96,8 @@ void FSEventWrap::GetInitialized(const FunctionCallbackInfo<Value>& args) {
 
 void FSEventWrap::Initialize(Local<Object> target,
                              Local<Value> unused,
-                             Local<Context> context) {
+                             Local<Context> context,
+                             void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   auto fsevent_string = FIXED_ONE_BYTE_STRING(env->isolate(), "FSEvent");

diff --git a/src/heap_utils.cc b/src/heap_utils.cc
@@ -247,7 +247,8 @@ void CreateHeapDump(const FunctionCallbackInfo<Value>& args) {
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   env->SetMethodNoSideEffect(target, "buildEmbedderGraph", BuildEmbedderGraph);

diff --git a/src/js_stream.cc b/src/js_stream.cc
@@ -196,7 +196,8 @@ void JSStream::EmitEOF(const FunctionCallbackInfo<Value>& args) {
 
 void JSStream::Initialize(Local<Object> target,
                           Local<Value> unused,
-                          Local<Context> context) {
+                          Local<Context> context,
+                          void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   Local<FunctionTemplate> t = env->NewFunctionTemplate(New);

diff --git a/src/js_stream.h b/src/js_stream.h
@@ -14,7 +14,8 @@ class JSStream : public AsyncWrap, public StreamBase {
  public:
   static void Initialize(v8::Local<v8::Object> target,
                          v8::Local<v8::Value> unused,
-                         v8::Local<v8::Context> context);
+                         v8::Local<v8::Context> context,
+                         void* priv);
 
   bool IsAlive() override;
   bool IsClosing() override;

diff --git a/src/module_wrap.cc b/src/module_wrap.cc
@@ -821,7 +821,8 @@ void ModuleWrap::SetInitializeImportMetaObjectCallback(
 
 void ModuleWrap::Initialize(Local<Object> target,
                             Local<Value> unused,
-                            Local<Context> context) {
+                            Local<Context> context,
+                            void* priv) {
   Environment* env = Environment::GetCurrent(context);
   Isolate* isolate = env->isolate();
 

diff --git a/src/module_wrap.h b/src/module_wrap.h
@@ -38,7 +38,8 @@ class ModuleWrap : public BaseObject {
   static const std::string EXTENSIONS[];
   static void Initialize(v8::Local<v8::Object> target,
                          v8::Local<v8::Value> unused,
-                         v8::Local<v8::Context> context);
+                         v8::Local<v8::Context> context,
+                         void* priv);
   static void HostInitializeImportMetaObjectCallback(
       v8::Local<v8::Context> context,
       v8::Local<v8::Module> module,

diff --git a/src/node_buffer.cc b/src/node_buffer.cc
@@ -1085,7 +1085,8 @@ void SetupBufferJS(const FunctionCallbackInfo<Value>& args) {
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   env->SetMethod(target, "setupBufferJS", SetupBufferJS);

diff --git a/src/node_contextify.cc b/src/node_contextify.cc
@@ -1075,7 +1075,8 @@ void ContextifyContext::CompileFunction(
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
   ContextifyContext::Init(env, target);
   ContextifyScript::Init(env, target);

diff --git a/src/node_domain.cc b/src/node_domain.cc
@@ -22,7 +22,8 @@ void Enable(const FunctionCallbackInfo<Value>& args) {
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   env->SetMethod(target, "enable", Enable);

diff --git a/src/node_native_module.cc b/src/node_native_module.cc
@@ -317,7 +317,8 @@ MaybeLocal<Value> NativeModuleLoader::LookupAndCompile(
 
 void NativeModuleLoader::Initialize(Local<Object> target,
                                     Local<Value> unused,
-                                    Local<Context> context) {
+                                    Local<Context> context,
+                                    void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   env->SetMethod(

diff --git a/src/node_native_module.h b/src/node_native_module.h
@@ -37,7 +37,8 @@ class NativeModuleLoader {
   NativeModuleLoader();
   static void Initialize(v8::Local<v8::Object> target,
                          v8::Local<v8::Value> unused,
-                         v8::Local<v8::Context> context);
+                         v8::Local<v8::Context> context,
+                         void* priv);
   v8::Local<v8::Object> GetSourceObject(v8::Local<v8::Context> context) const;
   v8::Local<v8::String> GetSource(v8::Isolate* isolate, const char* id) const;
 

diff --git a/src/node_options.cc b/src/node_options.cc
@@ -483,7 +483,8 @@ void GetOptions(const FunctionCallbackInfo<Value>& args) {
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
   Isolate* isolate = env->isolate();
   env->SetMethodNoSideEffect(target, "getOptions", GetOptions);

diff --git a/src/node_os.cc b/src/node_os.cc
@@ -446,7 +446,8 @@ static void GetPriority(const FunctionCallbackInfo<Value>& args) {
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
   env->SetMethod(target, "getHostname", GetHostname);
   env->SetMethod(target, "getLoadAvg", GetLoadAvg);

diff --git a/src/node_perf.cc b/src/node_perf.cc
@@ -376,7 +376,8 @@ void Timerify(const FunctionCallbackInfo<Value>& args) {
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
   Isolate* isolate = env->isolate();
   performance_state* state = env->performance_state();

diff --git a/src/node_serdes.cc b/src/node_serdes.cc
@@ -440,7 +440,8 @@ void DeserializerContext::ReadRawBytes(
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
   Local<FunctionTemplate> ser =
       env->NewFunctionTemplate(SerializerContext::New);

diff --git a/src/node_trace_events.cc b/src/node_trace_events.cc
@@ -21,6 +21,11 @@ using v8::Value;
 
 class NodeCategorySet : public BaseObject {
  public:
+  static void Initialize(Local<Object> target,
+                  Local<Value> unused,
+                  Local<Context> context,
+                  void* priv);
+
   static void New(const FunctionCallbackInfo<Value>& args);
   static void Enable(const FunctionCallbackInfo<Value>& args);
   static void Disable(const FunctionCallbackInfo<Value>& args);
@@ -97,7 +102,7 @@ void GetEnabledCategories(const FunctionCallbackInfo<Value>& args) {
   }
 }
 
-void Initialize(Local<Object> target,
+void NodeCategorySet::Initialize(Local<Object> target,
                 Local<Value> unused,
                 Local<Context> context,
                 void* priv) {
@@ -136,4 +141,5 @@ void Initialize(Local<Object> target,
 
 }  // namespace node
 
-NODE_MODULE_CONTEXT_AWARE_INTERNAL(trace_events, node::Initialize)
+NODE_MODULE_CONTEXT_AWARE_INTERNAL(trace_events,
+                                   node::NodeCategorySet::Initialize)
diff --git a/src/node_types.cc b/src/node_types.cc
@@ -62,7 +62,8 @@ static void IsBoxedPrimitive(const FunctionCallbackInfo<Value>& args) {
 
 void InitializeTypes(Local<Object> target,
                      Local<Value> unused,
-                     Local<Context> context) {
+                     Local<Context> context,
+                     void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
 #define V(type) env->SetMethodNoSideEffect(target,     \

diff --git a/src/node_util.cc b/src/node_util.cc
@@ -56,6 +56,7 @@ static void GetOwnNonIndexProperties(
 }
 
 static void GetPromiseDetails(const FunctionCallbackInfo<Value>& args) {
+  Environment* env = Environment::GetCurrent(args);
   // Return undefined if it's not a Promise.
   if (!args[0]->IsPromise())
     return;
@@ -74,6 +75,7 @@ static void GetPromiseDetails(const FunctionCallbackInfo<Value>& args) {
 }
 
 static void GetProxyDetails(const FunctionCallbackInfo<Value>& args) {
+  Environment* env = Environment::GetCurrent(args);
   // Return undefined if it's not a proxy.
   if (!args[0]->IsProxy())
     return;
@@ -194,7 +196,8 @@ void EnqueueMicrotask(const FunctionCallbackInfo<Value>& args) {
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
 #define V(name, _)                                                            \

diff --git a/src/node_v8.cc b/src/node_v8.cc
@@ -119,7 +119,8 @@ void SetFlagsFromString(const FunctionCallbackInfo<Value>& args) {
 
 void Initialize(Local<Object> target,
                 Local<Value> unused,
-                Local<Context> context) {
+                Local<Context> context,
+                void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   env->SetMethodNoSideEffect(target, "cachedDataVersionTag",

diff --git a/src/pipe_wrap.cc b/src/pipe_wrap.cc
@@ -69,7 +69,8 @@ Local<Object> PipeWrap::Instantiate(Environment* env,
 
 void PipeWrap::Initialize(Local<Object> target,
                           Local<Value> unused,
-                          Local<Context> context) {
+                          Local<Context> context,
+                          void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   Local<FunctionTemplate> t = env->NewFunctionTemplate(New);

diff --git a/src/pipe_wrap.h b/src/pipe_wrap.h
@@ -43,7 +43,8 @@ class PipeWrap : public ConnectionWrap<PipeWrap, uv_pipe_t> {
                                            SocketType type);
   static void Initialize(v8::Local<v8::Object> target,
                          v8::Local<v8::Value> unused,
-                         v8::Local<v8::Context> context);
+                         v8::Local<v8::Context> context,
+                         void* priv);
 
   SET_NO_MEMORY_INFO()
   SET_MEMORY_INFO_NAME(PipeWrap)

diff --git a/src/process_wrap.cc b/src/process_wrap.cc
@@ -49,7 +49,8 @@ class ProcessWrap : public HandleWrap {
  public:
   static void Initialize(Local<Object> target,
                          Local<Value> unused,
-                         Local<Context> context) {
+                         Local<Context> context,
+                         void* priv) {
     Environment* env = Environment::GetCurrent(context);
     Local<FunctionTemplate> constructor = env->NewFunctionTemplate(New);
     constructor->InstanceTemplate()->SetInternalFieldCount(1);

diff --git a/src/signal_wrap.cc b/src/signal_wrap.cc
@@ -43,7 +43,8 @@ class SignalWrap : public HandleWrap {
  public:
   static void Initialize(Local<Object> target,
                          Local<Value> unused,
-                         Local<Context> context) {
+                         Local<Context> context,
+                         void* priv) {
     Environment* env = Environment::GetCurrent(context);
     Local<FunctionTemplate> constructor = env->NewFunctionTemplate(New);
     constructor->InstanceTemplate()->SetInternalFieldCount(1);

diff --git a/src/spawn_sync.cc b/src/spawn_sync.cc
@@ -365,7 +365,8 @@ void SyncProcessStdioPipe::CloseCallback(uv_handle_t* handle) {
 
 void SyncProcessRunner::Initialize(Local<Object> target,
                                    Local<Value> unused,
-                                   Local<Context> context) {
+                                   Local<Context> context,
+                                   void* priv) {
   Environment* env = Environment::GetCurrent(context);
   env->SetMethod(target, "spawn", Spawn);
 }

diff --git a/src/spawn_sync.h b/src/spawn_sync.h
@@ -141,7 +141,8 @@ class SyncProcessRunner {
  public:
   static void Initialize(v8::Local<v8::Object> target,
                          v8::Local<v8::Value> unused,
-                         v8::Local<v8::Context> context);
+                         v8::Local<v8::Context> context,
+                         void* priv);
   static void Spawn(const v8::FunctionCallbackInfo<v8::Value>& args);
 
  private:

diff --git a/src/stream_pipe.cc b/src/stream_pipe.cc
@@ -248,7 +248,8 @@ namespace {
 
 void InitializeStreamPipe(Local<Object> target,
                           Local<Value> unused,
-                          Local<Context> context) {
+                          Local<Context> context,
+                          void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   // Create FunctionTemplate for FileHandle::CloseReq

diff --git a/src/stream_wrap.cc b/src/stream_wrap.cc
@@ -52,7 +52,8 @@ using v8::Value;
 
 void LibuvStreamWrap::Initialize(Local<Object> target,
                                  Local<Value> unused,
-                                 Local<Context> context) {
+                                 Local<Context> context,
+                                 void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   auto is_construct_call_callback =

diff --git a/src/stream_wrap.h b/src/stream_wrap.h
@@ -37,7 +37,8 @@ class LibuvStreamWrap : public HandleWrap, public StreamBase {
  public:
   static void Initialize(v8::Local<v8::Object> target,
                          v8::Local<v8::Value> unused,
-                         v8::Local<v8::Context> context);
+                         v8::Local<v8::Context> context,
+                         void* priv);
 
   int GetFD() override;
   bool IsAlive() override;

diff --git a/src/string_decoder.cc b/src/string_decoder.cc
@@ -276,7 +276,8 @@ void FlushData(const FunctionCallbackInfo<Value>& args) {
 
 void InitializeStringDecoder(Local<Object> target,
                              Local<Value> unused,
-                             Local<Context> context) {
+                             Local<Context> context,
+                             void* priv) {
   Environment* env = Environment::GetCurrent(context);
   Isolate* isolate = env->isolate();
 

diff --git a/src/tcp_wrap.cc b/src/tcp_wrap.cc
@@ -73,7 +73,8 @@ Local<Object> TCPWrap::Instantiate(Environment* env,
 
 void TCPWrap::Initialize(Local<Object> target,
                          Local<Value> unused,
-                         Local<Context> context) {
+                         Local<Context> context,
+                         void* priv) {
   Environment* env = Environment::GetCurrent(context);
 
   Local<FunctionTemplate> t = env->NewFunctionTemplate(New);

diff --git a/src/tcp_wrap.h b/src/tcp_wrap.h
@@ -42,7 +42,8 @@ class TCPWrap : public ConnectionWrap<TCPWrap, uv_tcp_t> {
                                            SocketType type);
   static void Initialize(v8::Local<v8::Object> target,
                          v8::Local<v8::Value> unused,
-                         v8::Local<v8::Context> context);
+                         v8::Local<v8::Context> context,
+                         void* priv);
 
   SET_NO_MEMORY_INFO()
   SET_SELF_SIZE(TCPWrap)