nodejs
diff --git a/‎src/env-inl.h‎
Lines changed: 2 additions & 2 deletions b/‎src/env-inl.h‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎src/env.h‎
Lines changed: 7 additions & 4 deletions b/‎src/env.h‎
Lines changed: 7 additions & 4 deletions
@@ -1028,15 +1028,15 @@ inline void Environment::SetInstanceMethod(v8::Local<v8::FunctionTemplate> that,
   t->SetClassName(name_string);
 }
 
-void Environment::AddCleanupHook(void (*fn)(void*), void* arg) {
+void Environment::AddCleanupHook(CleanupCallback fn, void* arg) {
   auto insertion_info = cleanup_hooks_.emplace(CleanupHookCallback {
     fn, arg, cleanup_hook_counter_++
   });
   // Make sure there was no existing element with these values.
   CHECK_EQ(insertion_info.second, true);
 }
 
-void Environment::RemoveCleanupHook(void (*fn)(void*), void* arg) {
+void Environment::RemoveCleanupHook(CleanupCallback fn, void* arg) {
   CleanupHookCallback search { fn, arg, 0 };
   cleanup_hooks_.erase(search);
 }
 
@@ -891,7 +891,9 @@ class ShouldNotAbortOnUncaughtScope {
 
 class CleanupHookCallback {
  public:
-  CleanupHookCallback(void (*fn)(void*),
+  typedef void (*Callback)(void*);
+
+  CleanupHookCallback(Callback fn,
                       void* arg,
                       uint64_t insertion_order_counter)
       : fn_(fn), arg_(arg), insertion_order_counter_(insertion_order_counter) {}
@@ -911,7 +913,7 @@ class CleanupHookCallback {
 
  private:
   friend class Environment;
-  void (*fn_)(void*);
+  Callback fn_;
   void* arg_;
 
   // We keep track of the insertion order for these objects, so that we can
@@ -1319,8 +1321,9 @@ class Environment : public MemoryRetainer {
   void ScheduleTimer(int64_t duration);
   void ToggleTimerRef(bool ref);
 
-  inline void AddCleanupHook(void (*fn)(void*), void* arg);
-  inline void RemoveCleanupHook(void (*fn)(void*), void* arg);
+  using CleanupCallback = CleanupHookCallback::Callback;
+  inline void AddCleanupHook(CleanupCallback cb, void* arg);
+  inline void RemoveCleanupHook(CleanupCallback cb, void* arg);
   void RunCleanup();
 
   static size_t NearHeapLimitCallback(void* data,
Original file line number	Diff line number	Diff line change
@@ -1028,15 +1028,15 @@ inline void Environment::SetInstanceMethod(v8::Local<v8::FunctionTemplate> that,
1028	1028	t->SetClassName(name_string);
1029	1029	}
1030	1030
1031		- void Environment::AddCleanupHook(void (fn)(void), void* arg) {
	1031	+ void Environment::AddCleanupHook(CleanupCallback fn, void* arg) {
1032	1032	auto insertion_info = cleanup_hooks_.emplace(CleanupHookCallback {
1033	1033	fn, arg, cleanup_hook_counter_++
1034	1034	});
1035	1035	// Make sure there was no existing element with these values.
1036	1036	CHECK_EQ(insertion_info.second, true);
1037	1037	}
1038	1038
1039		- void Environment::RemoveCleanupHook(void (fn)(void), void* arg) {
	1039	+ void Environment::RemoveCleanupHook(CleanupCallback fn, void* arg) {
1040	1040	CleanupHookCallback search { fn, arg, 0 };
1041	1041	cleanup_hooks_.erase(search);
1042	1042	}